From 7dd49acf4b996255b9599d4fe3309d78f66a74cb Mon Sep 17 00:00:00 2001 From: Berthold Stoeger Date: Sun, 22 Jul 2018 12:21:58 +0200 Subject: Cleanup: remove DiveItem::icon_names member array Each DiveItem (which is a wrapper around diveId with some virtual functions), had a member icon_names, which is an array of four QStrings. These were not used anywhere and must be an obscure oversight and was probably planned as a static cons array?. In any case, remove it. There *was* a function-local analogous icon_names array in DiveItem::data() though. This array would initialize four QStrings from C-string literals on every invocation. Make this array static, local to the translation unit and use the QStringLiteral macro to construct the QString object at compile-time. Signed-off-by: Berthold Stoeger --- qt-models/divetripmodel.h | 1 - 1 file changed, 1 deletion(-) (limited to 'qt-models/divetripmodel.h') diff --git a/qt-models/divetripmodel.h b/qt-models/divetripmodel.h index 98dc5bca3..71a316074 100644 --- a/qt-models/divetripmodel.h +++ b/qt-models/divetripmodel.h @@ -47,7 +47,6 @@ public: QString displayTags() const; int countPhotos(dive *dive) const; int weight() const; - QString icon_names[4]; }; struct TripItem : public TreeItem { -- cgit v1.2.3-70-g09d2