From 2248069e2afe0af9f923a3da47481a91291b6b8b Mon Sep 17 00:00:00 2001
From: Claudio Cambra <claudio.cambra@nextcloud.com>
Date: Thu, 23 Feb 2023 16:10:47 +0100
Subject: [PATCH] Clean up declaration in SparkleUpdater header

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
---
 src/gui/updater/sparkleupdater.h | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/src/gui/updater/sparkleupdater.h b/src/gui/updater/sparkleupdater.h
index c3ccc5f01689..518e808cf04d 100644
--- a/src/gui/updater/sparkleupdater.h
+++ b/src/gui/updater/sparkleupdater.h
@@ -24,8 +24,11 @@ namespace OCC {
 class SparkleUpdater : public Updater
 {
     Q_OBJECT
+    Q_PROPERTY(QString statusString READ statusString NOTIFY statusChanged)
+    Q_PROPERTY(State state READ state NOTIFY statusChanged)
 
 public:
+    class SparkleInterface;
     enum class State {
         Unknown = 0,
         Idle,
@@ -33,22 +36,19 @@ class SparkleUpdater : public Updater
         AwaitingUserInput
     };
 
-    SparkleUpdater(const QUrl &appCastUrl);
-    ~SparkleUpdater();
+    explicit SparkleUpdater(const QUrl &appCastUrl);
+    ~SparkleUpdater() override;
 
-    static bool autoUpdaterAllowed();
+    [[nodiscard]] static bool autoUpdaterAllowed();
 
-    void setUpdateUrl(const QUrl &url);
+    [[nodiscard]] bool handleStartup() override { return false; }
+    [[nodiscard]] QString statusString() const;
+    [[nodiscard]] State state() const;
 
-    // unused in this updater
+public slots:
+    void setUpdateUrl(const QUrl &url);
     void checkForUpdate() override;
     void backgroundCheckForUpdate() override;
-    bool handleStartup() override { return false; }
-
-    QString statusString() const;
-    State state() const;
-
-    class SparkleInterface;
 
 signals:
     void statusChanged();