Changeset 89323 in vbox for trunk/src/VBox/Frontends/VirtualBox/src/settings/global/UIGlobalSettingsDisplay.cpp

Timestamp:

May 27, 2021 2:23:11 PM (4 years ago)

Author:

vboxsync

Message:

FE/Qt: bugref:9827: Cleanup in Global Preferences pages; Mostly to comply with coding-style standards around.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/settings/global/UIGlobalSettingsDisplay.cpp (modified) (6 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/settings/global/UIGlobalSettingsDisplay.cpp

@@ -5 +5 @@
 
 /*
- * Copyright (C) 2012-2020 Oracle Corporation
+ * Copyright (C) 2012-2021 Oracle Corporation
  *
  * This file is part of VirtualBox Open Source Edition (OSE), as
@@ -41 +41 @@
     bool equal(const UIDataSettingsGlobalDisplay &other) const
     {
-        return true
+        return    true
                && (m_guiMaximumGuestScreenSizeValue == other.m_guiMaximumGuestScreenSizeValue)
                && (m_fActivateHoveredMachineWindow == other.m_fActivateHoveredMachineWindow)
                && (m_scaleFactors == other.m_scaleFactors)
-               ;
+                  ;
     }
 
@@ -61 +61 @@
 };
 
+
+/*********************************************************************************************************************************
+*   Class UIGlobalSettingsDisplay implementation.                                                                                *
+*********************************************************************************************************************************/
 
 UIGlobalSettingsDisplay::UIGlobalSettingsDisplay()
@@ -129 +133 @@
 
     /* Update data and failing state: */
-    setFailed(!saveDisplayData());
+    setFailed(!saveData());
 
     /* Upload properties to data: */
@@ -240 +244 @@
 }
 
-bool UIGlobalSettingsDisplay::saveDisplayData()
+bool UIGlobalSettingsDisplay::saveData()
 {
     /* Prepare result: */
     bool fSuccess = true;
     /* Save display settings from cache: */
-    if (fSuccess && m_pCache->wasChanged())
+    if (   fSuccess
+        && m_pCache->wasChanged())
     {
         /* Get old data from cache: */
@@ -253 +258 @@
 
         /* Save maximum guest screen size and policy: */
-        if (fSuccess && newData.m_guiMaximumGuestScreenSizeValue != oldData.m_guiMaximumGuestScreenSizeValue)
-            gEDataManager->setMaxGuestScreenResolution(newData.m_guiMaximumGuestScreenSizeValue.m_enmPolicy,
-                                                       newData.m_guiMaximumGuestScreenSizeValue.m_size);
+        if (   fSuccess
+            && newData.m_guiMaximumGuestScreenSizeValue != oldData.m_guiMaximumGuestScreenSizeValue)
+            /* fSuccess = */ gEDataManager->setMaxGuestScreenResolution(newData.m_guiMaximumGuestScreenSizeValue.m_enmPolicy,
+                                                                        newData.m_guiMaximumGuestScreenSizeValue.m_size);
         /* Save whether hovered machine-window should be activated automatically: */
-        if (fSuccess && newData.m_fActivateHoveredMachineWindow != oldData.m_fActivateHoveredMachineWindow)
-            gEDataManager->setActivateHoveredMachineWindow(newData.m_fActivateHoveredMachineWindow);
+        if (   fSuccess
+            && newData.m_fActivateHoveredMachineWindow != oldData.m_fActivateHoveredMachineWindow)
+            /* fSuccess = */ gEDataManager->setActivateHoveredMachineWindow(newData.m_fActivateHoveredMachineWindow);
         /* Save guest-screen scale-factor: */
-        if (fSuccess && newData.m_scaleFactors != oldData.m_scaleFactors)
-            gEDataManager->setScaleFactors(newData.m_scaleFactors, UIExtraDataManager::GlobalID);
+        if (   fSuccess
+            && newData.m_scaleFactors != oldData.m_scaleFactors)
+            /* fSuccess = */ gEDataManager->setScaleFactors(newData.m_scaleFactors, UIExtraDataManager::GlobalID);
     }
     /* Return result: */