From d7ec43633fcf1e13ed23d525319fc0cc1dd752d1 Mon Sep 17 00:00:00 2001 From: Mark Liversedge Date: Mon, 11 Feb 2013 11:32:23 +0000 Subject: [PATCH] Code Cleanup: Valgrind Memory Not sure why, but setting the text on a treewidget item seems to cost a lot of memory. So we don't bother since it is never used. In general the valgrind output is ok, quite surprised. There are very few non-widget items that are alloced but never free'd in the GC code. --- src/RideItem.cpp | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/src/RideItem.cpp b/src/RideItem.cpp index 8de48593f..766ad8393 100644 --- a/src/RideItem.cpp +++ b/src/RideItem.cpp @@ -31,13 +31,7 @@ RideItem::RideItem(int type, const Zones *zones, const HrZones *hrZones, MainWindow *main) : QTreeWidgetItem(type), ride_(NULL), main(main), isdirty(false), isedit(false), path(path), fileName(fileName), dateTime(dateTime), zones(zones), hrZones(hrZones) -{ - setText(0, dateTime.toString("ddd")); - setText(1, dateTime.toString("MMM d, yyyy")); - setText(2, dateTime.toString("h:mm AP")); - setTextAlignment(1, Qt::AlignRight); - setTextAlignment(2, Qt::AlignRight); -} +{ } RideFile *RideItem::ride() {