Remove Q_REQUIRED_RESULT from method with side-effect where result not always require...

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

diff --git a/src/gui/sharemanager.h b/src/gui/sharemanager.h
index 1d9918f0451a951fa7f4038c6315b4d090c9013d..d5cff7c4699ea4bf372ffc5f722bc3b8fa35e9c0 100644 (file)
--- a/src/gui/sharemanager.h
+++ b/src/gui/sharemanager.h
@@ -115,7 +115,10 @@ public:
      */
     [[nodiscard]] Permissions getPermissions() const;
 
-    [[nodiscard]] bool isPasswordSet() const;
+    /*
+     * Get whether the share has a password set
+     */
+    [[nodiscard]] Q_REQUIRED_RESULT bool isPasswordSet() const;
 
      /*
      * Is it a share with a user or group (local or remote)

diff --git a/src/gui/systray.h b/src/gui/systray.h
index d890f9fb420026dcfd112e96b6f347eb26479a6b..3578a977f5692d19956e500e6a9fc78bb1682d7d 100644 (file)
--- a/src/gui/systray.h
+++ b/src/gui/systray.h
@@ -91,7 +91,7 @@ public:
     Q_REQUIRED_RESULT bool syncIsPaused() const;
     Q_REQUIRED_RESULT bool isOpen() const;
 
-    Q_REQUIRED_RESULT bool raiseDialogs();
+    bool raiseDialogs();
 
 signals:
     void currentUserChanged();