[ros-diffs] [bfreisen] 73055: [MSPAINT] refactor RegistrySettings::SetWallpaper, also changing its signature to make it more usable

[bfreisen]

Author: bfreisen
Date: Fri Oct 28 19:48:25 2016
New Revision: 73055

URL: http://svn.reactos.org/svn/reactos?rev=73055&view=rev
Log:
[MSPAINT] refactor RegistrySettings::SetWallpaper, also changing its signature 
to make it more usable

Modified:
    trunk/reactos/base/applications/mspaint/registry.cpp
    trunk/reactos/base/applications/mspaint/registry.h
    trunk/reactos/base/applications/mspaint/winproc.cpp

Modified: trunk/reactos/base/applications/mspaint/registry.cpp
URL: 
http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint/registry.cpp?rev=73055&r1=73054&r2=73055&view=diff
==============================================================================
--- trunk/reactos/base/applications/mspaint/registry.cpp        [iso-8859-1] 
(original)
+++ trunk/reactos/base/applications/mspaint/registry.cpp        [iso-8859-1] 
Fri Oct 28 19:48:25 2016
@@ -35,23 +35,15 @@
     strFile.ReleaseBuffer();
 }
 
-void RegistrySettings::SetWallpaper(LPCTSTR szFileName, DWORD dwStyle, DWORD 
dwTile)
+void RegistrySettings::SetWallpaper(LPCTSTR szFileName, 
RegistrySettings::WallpaperStyle style)
 {
-    if ((dwStyle > 2) || (dwTile > 2))
-        return;
-
     CRegKey desktop;
     if (desktop.Open(HKEY_CURRENT_USER, _T("Control Panel\\Desktop")) == 
ERROR_SUCCESS)
     {
-        CString strStyle, strTile;
-
         desktop.SetStringValue(_T("Wallpaper"), szFileName);
 
-        strStyle.Format(_T("%lu"), dwStyle);
-        strTile.Format(_T("%lu"), dwTile);
-
-        desktop.SetStringValue(_T("WallpaperStyle"), strStyle);
-        desktop.SetStringValue(_T("TileWallpaper"), strTile);
+        desktop.SetStringValue(_T("WallpaperStyle"), (style == 
RegistrySettings::STRETCHED) ? _T("2") : _T("1"));
+        desktop.SetStringValue(_T("TileWallpaper"), (style == 
RegistrySettings::TILED) ? _T("1") : _T("0"));
     }
 
     SystemParametersInfo(SPI_SETDESKWALLPAPER, 0, (PVOID) szFileName, 
SPIF_UPDATEINIFILE | SPIF_SENDWININICHANGE);

Modified: trunk/reactos/base/applications/mspaint/registry.h
URL: 
http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint/registry.h?rev=73055&r1=73054&r2=73055&view=diff
==============================================================================
--- trunk/reactos/base/applications/mspaint/registry.h  [iso-8859-1] (original)
+++ trunk/reactos/base/applications/mspaint/registry.h  [iso-8859-1] Fri Oct 28 
19:48:25 2016
@@ -30,7 +30,13 @@
     CString strFile3;
     CString strFile4;
 
-    static void SetWallpaper(LPCTSTR szFileName, DWORD dwStyle, DWORD dwTile);
+    enum WallpaperStyle {
+        TILED,
+        CENTERED,
+        STRETCHED
+    };
+
+    static void SetWallpaper(LPCTSTR szFileName, WallpaperStyle style);
 
     void Load();
     void Store();

Modified: trunk/reactos/base/applications/mspaint/winproc.cpp
URL: 
http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint/winproc.cpp?rev=73055&r1=73054&r2=73055&view=diff
==============================================================================
--- trunk/reactos/base/applications/mspaint/winproc.cpp [iso-8859-1] (original)
+++ trunk/reactos/base/applications/mspaint/winproc.cpp [iso-8859-1] Fri Oct 28 
19:48:25 2016
@@ -451,13 +451,13 @@
                 GlobalFree(pd.hDevNames);
             break;
         case IDM_FILEASWALLPAPERPLANE:
-            RegistrySettings::SetWallpaper(filepathname, 1, 1);
+            RegistrySettings::SetWallpaper(filepathname, 
RegistrySettings::TILED);
             break;
         case IDM_FILEASWALLPAPERCENTERED:
-            RegistrySettings::SetWallpaper(filepathname, 1, 0);
+            RegistrySettings::SetWallpaper(filepathname, 
RegistrySettings::CENTERED);
             break;
         case IDM_FILEASWALLPAPERSTRETCHED:
-            RegistrySettings::SetWallpaper(filepathname, 2, 0);
+            RegistrySettings::SetWallpaper(filepathname, 
RegistrySettings::STRETCHED);
             break;
         case IDM_FILE1:
         {