[MAGNIFY] Minor whitespace / formatting fixes; use the new file headers + some code...
[reactos.git] / base / applications / magnify / settings.c
index ab6c585..a5d57be 100644 (file)
@@ -1,10 +1,9 @@
 /*
  * PROJECT:     ReactOS Magnify
- * LICENSE:     GPL - See COPYING in the top level directory
- * FILE:        base/applications/magnify/settings.c
+ * LICENSE:     GPL-2.0+ (https://spdx.org/licenses/GPL-2.0+)
  * PURPOSE:     Magnification of parts of the screen.
- * COPYRIGHT:   Copyright 2007 Marc Piulachs <marc.piulachs@codexchange.net>
- *
+ * COPYRIGHT:   Copyright 2007-2019 Marc Piulachs <marc.piulachs@codexchange.net>
+ *              Copyright 2015-2019 David Quintana <gigaherz@gmail.com>
  */
 
 #include "magnifier.h"
@@ -24,7 +23,8 @@ BOOL bInvertColors = FALSE;
 BOOL bStartMinimized = FALSE;
 BOOL bShowMagnifier = TRUE;
 
-struct _AppBarConfig_t AppBarConfig = {
+struct _AppBarConfig_t AppBarConfig =
+{
     sizeof(struct _AppBarConfig_t),
     -2 /* ABE_TOP */,
     0, 1, /* unknown */
@@ -32,96 +32,96 @@ struct _AppBarConfig_t AppBarConfig = {
     { 20, 20, 600, 200 }, /* floating window rect */
 };
 
-void LoadSettings()
+void LoadSettings(void)
 {
     HKEY hkey;
     LONG value;
     ULONG len;
     struct _AppBarConfig_t config_temp;
 
-    if (RegOpenKeyEx(HKEY_CURRENT_USER, _T("Software\\Microsoft\\Magnify"), 0, KEY_READ, &hkey) == ERROR_SUCCESS)
+    if (RegOpenKeyEx(HKEY_CURRENT_USER, _T("Software\\Microsoft\\Magnify"), 0, KEY_READ, &hkey) != ERROR_SUCCESS)
+        return;
+
+    len = sizeof(AppBarConfig);
+    if (RegQueryValueEx(hkey, _T("AppBar"), 0, 0, (BYTE *)&config_temp, &len) == ERROR_SUCCESS)
     {
-        len = sizeof(AppBarConfig);
-        if (RegQueryValueEx(hkey, _T("AppBar"), 0, 0, (BYTE *)&config_temp, &len) == ERROR_SUCCESS)
+        if(config_temp.cbSize == sizeof(AppBarConfig))
         {
-            if(config_temp.cbSize == sizeof(AppBarConfig))
-            {
-                AppBarConfig = config_temp;
-            }
+            AppBarConfig = config_temp;
         }
+    }
 
-        len = sizeof(value);
-        if (RegQueryValueEx(hkey, _T("StationaryMagLevel"),  0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
-        {
-            if (value >= 0 && value <= 9)
-                uiZoom = value;
-        }
+    len = sizeof(value);
+    if (RegQueryValueEx(hkey, _T("StationaryMagLevel"), 0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
+    {
+        if (value >= 0 && value <= 9)
+            uiZoom = value;
+    }
 
-        len = sizeof(value);
-        if (RegQueryValueEx(hkey, _T("ShowWarning"), 0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
-            bShowWarning = (value == 0 ? FALSE : TRUE);
+    len = sizeof(value);
+    if (RegQueryValueEx(hkey, _T("ShowWarning"), 0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
+        bShowWarning = !!value;
 
-        len = sizeof(value);
-        if (RegQueryValueEx(hkey, _T("StationaryInvertColors"), 0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
-            bInvertColors = (value == 0 ? FALSE : TRUE);
+    len = sizeof(value);
+    if (RegQueryValueEx(hkey, _T("StationaryInvertColors"), 0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
+        bInvertColors = !!value;
 
-        len = sizeof(value);
-        if (RegQueryValueEx(hkey, _T("StationaryStartMinimized"), 0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
-            bStartMinimized = (value == 0 ? FALSE : TRUE);
+    len = sizeof(value);
+    if (RegQueryValueEx(hkey, _T("StationaryStartMinimized"), 0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
+        bStartMinimized = !!value;
 
-        len = sizeof(value);
-        if (RegQueryValueEx(hkey, _T("StationaryTrackCursor"), 0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
-            bFollowMouse = (value == 0 ? FALSE : TRUE);
+    len = sizeof(value);
+    if (RegQueryValueEx(hkey, _T("StationaryTrackCursor"), 0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
+        bFollowMouse = !!value;
 
-        len = sizeof(value);
-        if (RegQueryValueEx(hkey, _T("StationaryTrackFocus"), 0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
-            bFollowFocus = (value == 0 ? FALSE : TRUE);
+    len = sizeof(value);
+    if (RegQueryValueEx(hkey, _T("StationaryTrackFocus"), 0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
+        bFollowFocus = !!value;
 
-        len = sizeof(value);
-        if (RegQueryValueEx(hkey, _T("StationaryTrackSecondaryFocus"), 0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
-            bFollowFocus = (value == 0 ? FALSE : TRUE);
+    len = sizeof(value);
+    if (RegQueryValueEx(hkey, _T("StationaryTrackSecondaryFocus"), 0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
+        bFollowFocus = !!value;
 
-        len = sizeof(value);
-        if (RegQueryValueEx(hkey, _T("StationaryTrackText"), 0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
-            bFollowCaret = (value == 0 ? FALSE : TRUE);
+    len = sizeof(value);
+    if (RegQueryValueEx(hkey, _T("StationaryTrackText"), 0, 0, (BYTE *)&value, &len) == ERROR_SUCCESS)
+        bFollowCaret = !!value;
 
-        RegCloseKey(hkey);
-    }
+    RegCloseKey(hkey);
 }
 
-void SaveSettings()
+void SaveSettings(void)
 {
     HKEY hkey;
     LONG value;
 
-    if (RegCreateKeyEx(HKEY_CURRENT_USER, _T("Software\\Microsoft\\Magnify"), 0, _T(""), 0, KEY_WRITE, NULL, &hkey, NULL) == ERROR_SUCCESS)
-    {
-        RegSetValueEx(hkey, _T("AppBar"), 0, REG_BINARY, (BYTE *)&AppBarConfig, sizeof(AppBarConfig));
+    if (RegCreateKeyEx(HKEY_CURRENT_USER, _T("Software\\Microsoft\\Magnify"), 0, _T(""), 0, KEY_WRITE, NULL, &hkey, NULL) != ERROR_SUCCESS)
+        return;
 
-        value = uiZoom;
-        RegSetValueEx(hkey, _T("StationaryMagLevel"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
+    RegSetValueEx(hkey, _T("AppBar"), 0, REG_BINARY, (BYTE *)&AppBarConfig, sizeof(AppBarConfig));
 
-        value = bShowWarning;
-        RegSetValueEx(hkey, _T("ShowWarning"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
+    value = uiZoom;
+    RegSetValueEx(hkey, _T("StationaryMagLevel"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
 
-        value = bInvertColors;
-        RegSetValueEx(hkey, _T("StationaryInvertColors"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
+    value = bShowWarning;
+    RegSetValueEx(hkey, _T("ShowWarning"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
 
-        value = bStartMinimized;
-        RegSetValueEx(hkey, _T("StationaryStartMinimized"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
+    value = bInvertColors;
+    RegSetValueEx(hkey, _T("StationaryInvertColors"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
 
-        value = bFollowMouse;
-        RegSetValueEx(hkey, _T("StationaryTrackCursor"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
+    value = bStartMinimized;
+    RegSetValueEx(hkey, _T("StationaryStartMinimized"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
 
-        value = bFollowFocus;
-        RegSetValueEx(hkey, _T("StationaryTrackFocus"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
+    value = bFollowMouse;
+    RegSetValueEx(hkey, _T("StationaryTrackCursor"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
 
-        value = bFollowFocus;
-        RegSetValueEx(hkey, _T("StationaryTrackSecondaryFocus"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
+    value = bFollowFocus;
+    RegSetValueEx(hkey, _T("StationaryTrackFocus"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
 
-        value = bFollowCaret;
-        RegSetValueEx(hkey, _T("StationaryTrackText"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
+    value = bFollowFocus;
+    RegSetValueEx(hkey, _T("StationaryTrackSecondaryFocus"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
 
-        RegCloseKey(hkey);
-    }
+    value = bFollowCaret;
+    RegSetValueEx(hkey, _T("StationaryTrackText"), 0, REG_DWORD, (BYTE *)&value, sizeof(value));
+
+    RegCloseKey(hkey);
 }