From a1365fc729d79a91eabad824404ec60d79d62138 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 | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/gui/updater/sparkleupdater.h b/src/gui/updater/sparkleupdater.h
index c3ccc5f01689..38c46e734525 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,
@@ -34,21 +37,18 @@ class SparkleUpdater : public Updater
     };
 
     SparkleUpdater(const QUrl &appCastUrl);
-    ~SparkleUpdater();
+    ~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();