Changeset 45329 in vbox for trunk/src

Timestamp:

Apr 4, 2013 9:00:44 AM (12 years ago)

Author:

vboxsync

Message:

FE/Qt: VM selector: Multi-target ACPI shutdown signal processing cleanup/rework.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/selector/UISelectorWindow.cpp (modified) (1 diff)

trunk/src/VBox/Frontends/VirtualBox/src/selector/UISelectorWindow.cpp

@@ -1805 +1805 @@
 bool UISelectorWindow::isAtLeastOneItemAbleToShutdown(const QList<UIVMItem*> &items)
 {
+    /* Enumerate all the passed items: */
     foreach (UIVMItem *pItem, items)
     {
+        /* Skip non-running machines: */
         if (!UIVMItem::isItemRunning(pItem))
             continue;
-
+        /* Skip session failures: */
         CSession session = vboxGlobal().openExistingSession(pItem->id());
         if (session.isNull())
-            return false;
+            continue;
+        /* Skip console failures: */
         CConsole console = session.GetConsole();
         if (console.isNull())
         {
+            /* Do not forget to release machine: */
             session.UnlockMachine();
-            return false;
-        }
+            continue;
+        }
+        /* Is the guest entered ACPI mode? */
+        bool fGuestEnteredACPIMode = console.GetGuestEnteredACPIMode();
+        /* Do not forget to release machine: */
         session.UnlockMachine();
-
-        return console.GetGuestEnteredACPIMode();
-    }
+        /* True if the guest entered ACPI mode: */
+        if (fGuestEnteredACPIMode)
+            return true;
+    }
+    /* False by default: */
     return false;
 }