[pim/trojita] src/Imap/Tasks: Move KeepMailBoxOpentask nonslots to public/private

Espen Sandøy Hustad <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit 3379f36bc0ee0ce7040b8d64068e0d7a5521deb4 by Espen Sandøy Hustad.
Committed on 03/08/2026 at 19:00.
Pushed by ehustad into branch 'master'.

Move KeepMailBoxOpentask nonslots to public/private

These are not actually used as slots, and they even return
values.

Fix a bunch of Clazy warning: slot arguments need to be
fully-qualified [-Wclazy-fully-qualified-moc-types].

M  +8    -7    src/Imap/Tasks/KeepMailboxOpenTask.h

https://invent.kde.org/pim/trojita/-/commit/3379f36bc0ee0ce7040b8d64068e0d7a5521deb4

diff --git a/src/Imap/Tasks/KeepMailboxOpenTask.h b/src/Imap/Tasks/KeepMailboxOpenTask.h
index 38376f307..cd6fb46c2 100644
--- a/src/Imap/Tasks/KeepMailboxOpenTask.h
+++ b/src/Imap/Tasks/KeepMailboxOpenTask.h
@@ -159,6 +159,12 @@ public:
 
     bool hasItsOwnActivity() const;
 
+    bool handleStateHelper(const Imap::Responses::State *const resp) override;
+    bool handleNumberResponse(const Imap::Responses::NumberResponse *const resp) override;
+    bool handleFlags(const Imap::Responses::Flags *const resp) override;
+    bool handleFetch(const Imap::Responses::Fetch *const resp) override;
+    bool handleVanished(const Imap::Responses::Vanished *const resp) override;
+
 private slots:
     void slotTaskDeleted(QObject *object);
 
@@ -175,13 +181,6 @@ private slots:
     /** @short The synchronization is done, let's start working now */
     void slotSyncHasCompleted() { perform(); }
 
-    bool handleNumberResponse(const Imap::Responses::NumberResponse *const resp) override;
-    bool handleFetch(const Imap::Responses::Fetch *const resp) override;
-    bool handleStateHelper(const Imap::Responses::State *const resp) override;
-    bool handleFlags(const Imap::Responses::Flags *const resp) override;
-    bool handleVanished(const Imap::Responses::Vanished *const resp) override;
-    bool handleResponseCodeInsideState(const Imap::Responses::State *const resp);
-
     void slotPerformNoop();
     void slotActivateTasks() { activateTasks(); }
     void slotFetchRequestedParts();
@@ -198,6 +197,8 @@ private slots:
     void syncingTimeout();
 
 private:
+    bool handleResponseCodeInsideState(const Imap::Responses::State *const resp);
+
     /** @short Activate the dependent tasks while also limiting the rate */
     void activateTasks();
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.