Make connectToPaletteSignal const

We don't really need to have the check fo paletteSignalsConnected

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

diff --git a/src/libsync/theme.cpp b/src/libsync/theme.cpp
index cb672c71750dd5a3106cd207cb2f199768af6b39..2db9e708ca6165577c42de465aa166b68a39a8f3 100644 (file)
--- a/src/libsync/theme.cpp
+++ b/src/libsync/theme.cpp
@@ -964,13 +964,10 @@ QColor Theme::defaultColor()
     return QColor{NEXTCLOUD_BACKGROUND_COLOR};
 }
 
-void Theme::connectToPaletteSignal()
+void Theme::connectToPaletteSignal() const
 {
-    if (!_paletteSignalsConnected) {
-        if (const auto ptr = qobject_cast<QGuiApplication*>(qApp)) {
-            connect(ptr->styleHints(), &QStyleHints::colorSchemeChanged, this, &Theme::darkModeChanged);
-            _paletteSignalsConnected = true;
-        }
+    if (const auto ptr = qobject_cast<QGuiApplication*>(qApp)) {
+        connect(ptr->styleHints(), &QStyleHints::colorSchemeChanged, this, &Theme::darkModeChanged, Qt::UniqueConnection);
     }
 }
 

diff --git a/src/libsync/theme.h b/src/libsync/theme.h
index a7e817de15ca94e44b43ed77b40c79264c0ddc4b..9e12ba709f51c5b2830111d469b151fd6ad5aa88 100644 (file)
--- a/src/libsync/theme.h
+++ b/src/libsync/theme.h
@@ -639,14 +639,13 @@ private:
     Theme &operator=(Theme const &);
 
     void updateMultipleOverrideServers();
-    void connectToPaletteSignal();
+    void connectToPaletteSignal() const;
 #if defined(Q_OS_WIN)
     QPalette reserveDarkPalette; // Windows 11 button and window dark colours
 #endif
 
     static Theme *_instance;
     bool _mono = false;
-    bool _paletteSignalsConnected = false;
 
     QString _overrideServerUrl;
     bool _forceOverrideServerUrl = false;