From 3b42de66dc469e8783df2a3ea08e90b310821668 Mon Sep 17 00:00:00 2001 From: Berthold Stoeger Date: Fri, 31 Jan 2020 09:42:09 +0100 Subject: Cleanup: remove QMLInterface::instance() QMLInterface::instance() is only used in one single place. This makes the whole notion of having a global instance of the object moot, isn't it? Simply make the object static to the function that uses it, which guarantees that the object will be created when the function is called. I.e. the same behavior is retained with less complexity. Signed-off-by: Berthold Stoeger --- mobile-widgets/qmlinterface.cpp | 10 ++-------- mobile-widgets/qmlinterface.h | 4 +--- 2 files changed, 3 insertions(+), 11 deletions(-) (limited to 'mobile-widgets') diff --git a/mobile-widgets/qmlinterface.cpp b/mobile-widgets/qmlinterface.cpp index 67d4196a3..4bf23dedf 100644 --- a/mobile-widgets/qmlinterface.cpp +++ b/mobile-widgets/qmlinterface.cpp @@ -81,16 +81,11 @@ QMLInterface::QMLInterface() this, &QMLInterface::display_variationsChanged); } -QMLInterface *QMLInterface::instance() -{ - static QMLInterface *self = new QMLInterface; - return self; -} - void QMLInterface::setup(QQmlContext *ct) { // Register interface class - ct->setContextProperty("Backend", QMLInterface::instance()); + static QMLInterface self; + ct->setContextProperty("Backend", &self); // Make enums available as types qmlRegisterUncreatableType("org.subsurfacedivelog.mobile",1,0,"Enums","Enum is not a type"); @@ -100,7 +95,6 @@ void QMLInterface::setup(QQmlContext *ct) diveSummary::summaryCalculation(0, 3); } - void QMLInterface::summaryCalculation(int primaryPeriod, int secondaryPeriod) { diveSummary::summaryCalculation(primaryPeriod, secondaryPeriod); diff --git a/mobile-widgets/qmlinterface.h b/mobile-widgets/qmlinterface.h index 3c6022daa..1aced7a9d 100644 --- a/mobile-widgets/qmlinterface.h +++ b/mobile-widgets/qmlinterface.h @@ -81,9 +81,7 @@ class QMLInterface : public QObject { Q_PROPERTY(QStringList diveSummaryText READ diveSummaryText NOTIFY diveSummaryTextChanged); public: - static QMLInterface *instance(); - - // function to do the needed setup and do connect of signal/signal + // function to do the needed setup static void setup(QQmlContext *ct); // Duplicated enums, these enums are properly defined in the C/C++ structure -- cgit v1.2.3-70-g09d2