From 127cd211375ff68c45d18eaab8954627ac8932fe Mon Sep 17 00:00:00 2001
From: "Lubomir I. Ivanov" <neolit123@gmail.com>
Date: Thu, 11 Jul 2013 12:41:50 +0300
Subject: Models: add the DiveItem::displayDate() helper

Add this helper so that we can easily wrap the required
memory release after calling get_dive_date_string()
(fixes a small memory leak).

Signed-off-by: Lubomir I. Ivanov <neolit123@gmail.com>
---
 qt-ui/models.cpp | 10 +++++++++-
 qt-ui/models.h   |  1 +
 2 files changed, 10 insertions(+), 1 deletion(-)

(limited to 'qt-ui')

diff --git a/qt-ui/models.cpp b/qt-ui/models.cpp
index ba325e149..80ccf0e73 100644
--- a/qt-ui/models.cpp
+++ b/qt-ui/models.cpp
@@ -971,7 +971,7 @@ QVariant DiveItem::data(int column, int role) const
 	case Qt::DisplayRole:
 		switch (column) {
 		case NR:		retVal = dive->number; break;
-		case DATE:		retVal = QString(get_dive_date_string(dive->when)); break;
+		case DATE:		retVal = displayDate(); break;
 		case DEPTH:		retVal = displayDepth(); break;
 		case DURATION:		retVal = displayDuration(); break;
 		case TEMPERATURE:	retVal = displayTemperature(); break;
@@ -996,6 +996,14 @@ QVariant DiveItem::data(int column, int role) const
 	return retVal;
 }
 
+QString DiveItem::displayDate() const
+{
+	char *buf = get_dive_date_string(dive->when);
+	QString date(buf);
+	free(buf);
+	return date;
+}
+
 QString DiveItem::displayDepth() const
 {
 	const int scale = 1000;
diff --git a/qt-ui/models.h b/qt-ui/models.h
index 439ff05df..bd1e60c0f 100644
--- a/qt-ui/models.h
+++ b/qt-ui/models.h
@@ -142,6 +142,7 @@ struct DiveItem : public TreeItem {
 	virtual QVariant data(int column, int role) const;
 	struct dive* dive;
 
+	QString displayDate() const;
 	QString displayDuration() const;
 	QString displayDepth() const;
 	QString displayTemperature() const;
-- 
cgit v1.2.3-70-g09d2