diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2018-07-22 12:21:58 +0200 |
|---|---|---|
| committer |  Lubomir I. Ivanov <neolit123@gmail.com> | 2018-07-23 17:05:15 +0300 |
| commit | 7dd49acf4b996255b9599d4fe3309d78f66a74cb (patch) | |
| tree | 8a46a695e1b338db9d40c09203872a313cc25f29 /qt-models/divetripmodel.h | |
| parent | be50ee2017b726fdb7fc0a023cd32d91a55350fc (diff) | |
| download | subsurface-7dd49acf4b996255b9599d4fe3309d78f66a74cb.tar.gz | |
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 <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'qt-models/divetripmodel.h')
| -rw-r--r-- | qt-models/divetripmodel.h | 1 |
1 files changed, 0 insertions, 1 deletions
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 { |