From 127cd211375ff68c45d18eaab8954627ac8932fe Mon Sep 17 00:00:00 2001 From: "Lubomir I. Ivanov" 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 --- qt-ui/models.cpp | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) (limited to 'qt-ui/models.cpp') 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; -- cgit v1.2.3-70-g09d2