qa: Simplify main window activation logic

Remove code duplication, avoid a warning about unreachable code.

diff --git a/src/uisupport/graphicalui.cpp b/src/uisupport/graphicalui.cpp
index 73affbf..33b0d87 100644 (file)
--- a/src/uisupport/graphicalui.cpp
+++ b/src/uisupport/graphicalui.cpp
@@ -144,38 +144,26 @@ bool GraphicalUi::eventFilter(QObject* obj, QEvent* event)
 
 bool GraphicalUi::checkMainWidgetVisibility(bool perform)
 {
+    bool needsActivation{true};
+
 #ifdef Q_OS_WIN
     // the problem is that we lose focus when the systray icon is activated
     // and we don't know the former active window
     // therefore we watch for activation event and use our stopwatch :)
     if (GetTickCount() - _dwTickCount < 300) {
         // we were active in the last 300ms -> hide it
-        if (perform)
-            minimizeRestore(false);
-        return false;
-    }
-    else {
-        if (perform)
-            minimizeRestore(true);
-        return true;
+        needsActivation = false;
     }
-
 #else
-
-    if (!mainWidget()->isVisible() || mainWidget()->isMinimized() || !mainWidget()->isActiveWindow()) {
-        if (perform)
-            minimizeRestore(true);
-        return true;
+    if (mainWidget()->isVisible() && !mainWidget()->isMinimized() && mainWidget()->isActiveWindow()) {
+        needsActivation = false;
     }
-    else {
-        if (perform)
-            minimizeRestore(false);
-        return false;
-    }
-
 #endif
 
-    return true;
+    if (perform) {
+        minimizeRestore(needsActivation);
+    }
+    return needsActivation;
 }
 
 bool GraphicalUi::isMainWidgetVisible()