[SHELL32] Deduplicate MAX_PROPERTY_SHEET_PAGE and AddPropSheetPageCallback(). #179
authorAmine Khaldi <amine.khaldi@reactos.org>
Mon, 25 Dec 2017 12:22:03 +0000 (13:22 +0100)
committerAmine Khaldi <amine.khaldi@reactos.org>
Mon, 25 Dec 2017 12:22:23 +0000 (13:22 +0100)
dll/win32/shell32/CShellLink.cpp
dll/win32/shell32/dialogs/drive.cpp
dll/win32/shell32/dialogs/folder_options.cpp
dll/win32/shell32/dialogs/fprop.cpp
dll/win32/shell32/folders/CRecycleBin.cpp
dll/win32/shell32/precomp.h

index 918c98e..5b7b84d 100644 (file)
@@ -144,7 +144,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(shell);
 
 #define SHLINK_LOCAL  0
 #define SHLINK_REMOTE 1
-#define MAX_PROPERTY_SHEET_PAGE 32
 
 /* link file formats */
 
index 3afbe80..679afe6 100644 (file)
@@ -21,8 +21,6 @@
 
 #include "precomp.h"
 
-#define MAX_PROPERTY_SHEET_PAGE 32
-
 WINE_DEFAULT_DEBUG_CHANNEL(shell);
 
 typedef struct
@@ -109,18 +107,6 @@ GetDefaultClusterSize(LPWSTR szFs, PDWORD pClusterSize, PULARGE_INTEGER TotalNum
     return TRUE;
 }
 
-static BOOL CALLBACK
-AddPropSheetPageCallback(HPROPSHEETPAGE hPage, LPARAM lParam)
-{
-    PROPSHEETHEADER *ppsh = (PROPSHEETHEADER *)lParam;
-    if (ppsh->nPages < MAX_PROPERTY_SHEET_PAGE)
-    {
-        ppsh->phpage[ppsh->nPages++] = hPage;
-        return TRUE;
-    }
-    return FALSE;
-}
-
 typedef struct _DRIVE_PROP_PAGE
 {
     LPCSTR resname;
index 45908ff..3bb3997 100644 (file)
@@ -23,8 +23,6 @@
 
 WINE_DEFAULT_DEBUG_CHANNEL (fprop);
 
-#define MAX_PROPERTY_SHEET_PAGE (32)
-
 /// Folder Options:
 /// CLASSKEY = HKEY_CLASSES_ROOT\CLSID\{6DFD7C5C-2451-11d3-A299-00C04F8EF6AF}
 /// DefaultIcon = %SystemRoot%\system32\SHELL32.dll,-210
index 989a9ce..784d6ec 100644 (file)
 
 #include "precomp.h"
 
-#define MAX_PROPERTY_SHEET_PAGE 32
-
 WINE_DEFAULT_DEBUG_CHANNEL(shell);
 
 EXTERN_C HPSXA WINAPI SHCreatePropSheetExtArrayEx(HKEY hKey, LPCWSTR pszSubKey, UINT max_iface, IDataObject *pDataObj);
 
-static BOOL CALLBACK
-AddPropSheetPageCallback(HPROPSHEETPAGE hPage, LPARAM lParam)
-{
-    PROPSHEETHEADERW *pHeader = (PROPSHEETHEADERW *)lParam;
-
-    if (pHeader->nPages < MAX_PROPERTY_SHEET_PAGE)
-    {
-        pHeader->phpage[pHeader->nPages++] = hPage;
-        return TRUE;
-    }
-
-    return FALSE;
-}
-
 static UINT
 LoadPropSheetHandlers(LPCWSTR pwszPath, PROPSHEETHEADERW *pHeader, UINT cMaxPages, HPSXA *phpsxa, IDataObject *pDataObj)
 {
index 51be966..fb8c1d8 100644 (file)
@@ -24,8 +24,6 @@
 #include <mmsystem.h>
 #include <ntquery.h>
 
-#define MAX_PROPERTY_SHEET_PAGE 32
-
 WINE_DEFAULT_DEBUG_CHANNEL(CRecycleBin);
 
 typedef struct
index dbea414..03166e3 100644 (file)
@@ -93,4 +93,22 @@ extern const GUID CLSID_UnixDosFolder;
 extern const GUID SHELL32_AdvtShortcutProduct;
 extern const GUID SHELL32_AdvtShortcutComponent;
 
+#define MAX_PROPERTY_SHEET_PAGE 32
+
+extern inline
+BOOL
+CALLBACK
+AddPropSheetPageCallback(HPROPSHEETPAGE hPage, LPARAM lParam)
+{
+    PROPSHEETHEADERW *pHeader = (PROPSHEETHEADERW *)lParam;
+
+    if (pHeader->nPages < MAX_PROPERTY_SHEET_PAGE)
+    {
+        pHeader->phpage[pHeader->nPages++] = hPage;
+        return TRUE;
+    }
+
+    return FALSE;
+}
+
 #endif /* _PRECOMP_H__ */