Move loading of QNetworkInformation to main()

The `QNetworkInformation` will be used in multiple places. Do the loading once before any uses.
owncloud · Jan 4, 2024 · 140276d · 140276d
1 parent 772a93a
commit 140276d
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/src/gui/accountstate.cpp b/src/gui/accountstate.cpp
@@ -115,8 +115,8 @@ AccountState::AccountState(AccountPtr account)
         Qt::QueuedConnection);
 
 #if QT_VERSION >= QT_VERSION_CHECK(6, 3, 0)
-    if (QNetworkInformation::loadDefaultBackend()) {
-        connect(QNetworkInformation::instance(), &QNetworkInformation::reachabilityChanged, this, [this](QNetworkInformation::Reachability reachability) {
+    if (QNetworkInformation *qNetInfo = QNetworkInformation::instance()) {
+        connect(qNetInfo, &QNetworkInformation::reachabilityChanged, this, [this](QNetworkInformation::Reachability reachability) {
             switch (reachability) {
             case QNetworkInformation::Reachability::Online:
                 [[fallthrough]];
@@ -136,8 +136,6 @@ AccountState::AccountState(AccountPtr account)
                 break;
             }
         });
-    } else {
-        qCWarning(lcAccountState) << "Failed to load QNetworkInformation";
     }
 #endif
     // as a fallback and to recover after server issues we also poll

diff --git a/src/gui/main.cpp b/src/gui/main.cpp
@@ -38,6 +38,7 @@
 #include <QCommandLineParser>
 #include <QLibraryInfo>
 #include <QMessageBox>
+#include <QNetworkInformation>
 #include <QProcess>
 #include <QTimer>
 #include <QTranslator>
@@ -448,6 +449,14 @@ int main(int argc, char **argv)
         return -1;
     }
 
+#if QT_VERSION >= QT_VERSION_CHECK(6, 3, 0)
+    if (!QNetworkInformation::loadDefaultBackend()) {
+        qCWarning(lcMain) << "Failed to load QNetworkInformation";
+    }
+#else
+    qCWarning(lcMain) << "QNetworkInformation is not available";
+#endif
+
     setupLogging(options);
 
     platform->setApplication(&app);