From d357cdd5f9998cc9c3696235d3b06ac4cb249c93 Mon Sep 17 00:00:00 2001
From: Chocobo1 <Chocobo1@users.noreply.github.com>
Date: Fri, 21 Jul 2023 00:57:43 +0800
Subject: [PATCH] Set power state to idle when deconstructing class

---
 src/gui/powermanagement/powermanagement.cpp | 23 ++++++++++++++-------
 src/gui/powermanagement/powermanagement.h   |  4 ++--
 2 files changed, 17 insertions(+), 10 deletions(-)

diff --git a/src/gui/powermanagement/powermanagement.cpp b/src/gui/powermanagement/powermanagement.cpp
index dfdfb6e54..2fbecec45 100644
--- a/src/gui/powermanagement/powermanagement.cpp
+++ b/src/gui/powermanagement/powermanagement.cpp
@@ -44,10 +44,15 @@
 
 PowerManagement::PowerManagement(QObject *parent)
     : QObject(parent)
-{
 #ifdef QBT_USES_DBUS
-    m_inhibitor = new PowerManagementInhibitor(this);
+    , m_inhibitor {new PowerManagementInhibitor(this)}
 #endif
+{
+}
+
+PowerManagement::~PowerManagement()
+{
+    setIdle();
 }
 
 void PowerManagement::setActivityState(const bool busy)
@@ -60,15 +65,16 @@ void PowerManagement::setActivityState(const bool busy)
 
 void PowerManagement::setBusy()
 {
-    if (m_busy) return;
+    if (m_busy)
+        return;
     m_busy = true;
 
 #ifdef Q_OS_WIN
-    SetThreadExecutionState(ES_CONTINUOUS | ES_SYSTEM_REQUIRED);
+    ::SetThreadExecutionState(ES_CONTINUOUS | ES_SYSTEM_REQUIRED);
 #elif defined(QBT_USES_DBUS)
     m_inhibitor->requestBusy();
 #elif defined(Q_OS_MACOS)
-    IOReturn success = IOPMAssertionCreateWithName(kIOPMAssertionTypeNoIdleSleep, kIOPMAssertionLevelOn
+    const IOReturn success = ::IOPMAssertionCreateWithName(kIOPMAssertionTypeNoIdleSleep, kIOPMAssertionLevelOn
         , tr("qBittorrent is active").toCFString(), &m_assertionID);
     if (success != kIOReturnSuccess)
         m_busy = false;
@@ -77,14 +83,15 @@ void PowerManagement::setBusy()
 
 void PowerManagement::setIdle()
 {
-    if (!m_busy) return;
+    if (!m_busy)
+        return;
     m_busy = false;
 
 #ifdef Q_OS_WIN
-    SetThreadExecutionState(ES_CONTINUOUS);
+    ::SetThreadExecutionState(ES_CONTINUOUS);
 #elif defined(QBT_USES_DBUS)
     m_inhibitor->requestIdle();
 #elif defined(Q_OS_MACOS)
-    IOPMAssertionRelease(m_assertionID);
+    ::IOPMAssertionRelease(m_assertionID);
 #endif
 }
diff --git a/src/gui/powermanagement/powermanagement.h b/src/gui/powermanagement/powermanagement.h
index 6fc42c679..39bbcf6d0 100644
--- a/src/gui/powermanagement/powermanagement.h
+++ b/src/gui/powermanagement/powermanagement.h
@@ -40,14 +40,14 @@
 class PowerManagementInhibitor;
 #endif
 
-class PowerManagement : public QObject
+class PowerManagement final : public QObject
 {
   Q_OBJECT
   Q_DISABLE_COPY_MOVE(PowerManagement)
 
 public:
   PowerManagement(QObject *parent = nullptr);
-  virtual ~PowerManagement() = default;
+  ~PowerManagement() override;
 
   void setActivityState(bool busy);