diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2019-09-22 14:33:33 +0200 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2019-09-25 13:35:30 -0700 |
| commit | 05200f9266a4e8e723450abb281c597a48da2e9f (patch) | |
| tree | 670d7b30f4b77f40e35253243f95fb219d5c7aea /qt-models/gpslistmodel.cpp | |
| parent | f20d6187f0135ef3cf7f583785be65012336a99d (diff) | |
| download | subsurface-05200f9266a4e8e723450abb281c597a48da2e9f.tar.gz | |
Cleanup: unify idiosyncratic singletons
The way we handle singletons in QML, QML insists on allocating the
objects. This leads to a very idiosyncratic way of handling
singletons: The global instance pointer is set in the constructor.
Unify all these by implementing a "SillySingleton" template. All
of the weird singleton-classes can derive from this template and
don't have to bother with reimplementing the instance() function
with all the safety-checks, etc.
This serves firstly as documentation but also improves debugging
as we will now see wanted and unwanted creation and destruction
of these weird singletons.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'qt-models/gpslistmodel.cpp')
| -rw-r--r-- | qt-models/gpslistmodel.cpp | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/qt-models/gpslistmodel.cpp b/qt-models/gpslistmodel.cpp index 8d874d67b..8a623bf00 100644 --- a/qt-models/gpslistmodel.cpp +++ b/qt-models/gpslistmodel.cpp @@ -3,11 +3,8 @@ #include "core/qthelper.h" #include <QVector> -GpsListModel *GpsListModel::m_instance = NULL; - GpsListModel::GpsListModel(QObject *parent) : QAbstractListModel(parent) { - m_instance = this; } void GpsListModel::update() @@ -62,9 +59,3 @@ QHash<int, QByteArray> GpsListModel::roleNames() const roles[GpsLongitudeRole] = "longitude"; return roles; } - -GpsListModel *GpsListModel::instance() -{ - return m_instance; -} - |