From: Claudio Cambra <claudio.cambra@nextcloud.com>
Date: Fri, 30 Sep 2022 23:16:52 +0000 (+0200)
Subject: Remove Q_REQUIRED_RESULT from method with side-effect where result not always require... 
X-Git-Tag: archive/raspbian/3.16.7-1_deb13u1+rpi1~1^2~12^2~11^2~169^2~20
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=f97f39375b7143bc52dce25436648a0317eb2fc9;p=nextcloud-desktop.git

Remove Q_REQUIRED_RESULT from method with side-effect where result not always required, and viceversa

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

diff --git a/src/gui/sharemanager.h b/src/gui/sharemanager.h
index 1d9918f04..d5cff7c46 100644
--- 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 d890f9fb4..3578a977f 100644
--- 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();