Hugh McMaster : regedit: Combine error() and error_code_messagebox().

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 6a8aea20acc35f15145e93b6d49dbb5997ab2d24
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=6a8aea20acc35f15145e93b6d49dbb5997ab2d24

Author: Hugh McMaster <hugh.mcmaster at outlook.com>
Date:   Wed Feb  1 12:56:51 2017 +0000

regedit: Combine error() and error_code_messagebox().

Signed-off-by: Hugh McMaster <hugh.mcmaster at outlook.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 programs/regedit/edit.c     | 23 +++++++++--------------
 programs/regedit/framewnd.c |  2 +-
 programs/regedit/main.h     |  2 +-
 3 files changed, 11 insertions(+), 16 deletions(-)

diff --git a/programs/regedit/edit.c b/programs/regedit/edit.c
index 7cac512..64ecfcb 100644
--- a/programs/regedit/edit.c
+++ b/programs/regedit/edit.c
@@ -76,20 +76,15 @@ static INT __cdecl messagebox(HWND hwnd, INT buttons, INT titleId, INT resId, ..
     return result;
 }
 
-void __cdecl error(HWND hwnd, INT resId, ...)
+void __cdecl error_code_messagebox(HWND hwnd, unsigned int msg_id, ...)
 {
     __ms_va_list ap;
 
-    __ms_va_start(ap, resId);
-    vmessagebox(hwnd, MB_OK | MB_ICONERROR, IDS_ERROR, resId, ap);
+    __ms_va_start(ap, msg_id);
+    vmessagebox(hwnd, MB_OK|MB_ICONERROR, IDS_ERROR, msg_id, ap);
     __ms_va_end(ap);
 }
 
-static void error_code_messagebox(HWND hwnd, unsigned int msg_id)
-{
-    vmessagebox(hwnd, MB_OK|MB_ICONERROR, IDS_ERROR, msg_id, NULL);
-}
-
 static BOOL change_dword_base(HWND hwndDlg, BOOL toHex)
 {
     static const WCHAR percent_u[] = {'%','u',0};
@@ -214,17 +209,17 @@ static LPWSTR read_value(HWND hwnd, HKEY hKey, LPCWSTR valueName, DWORD *lpType,
             *buffer = '\0';
             return buffer;
         }
-        error(hwnd, IDS_BAD_VALUE, valueName);
+        error_code_messagebox(hwnd, IDS_BAD_VALUE, valueName);
         goto done;
     }
     if ( *lpType == REG_DWORD ) valueDataLen = sizeof(DWORD);
     if (!(buffer = HeapAlloc(GetProcessHeap(), 0, valueDataLen+sizeof(WCHAR)))) {
-        error(hwnd, IDS_TOO_BIG_VALUE, valueDataLen);
+        error_code_messagebox(hwnd, IDS_TOO_BIG_VALUE, valueDataLen);
         goto done;
     }
     lRet = RegQueryValueExW(hKey, valueName, 0, 0, (LPBYTE)buffer, &valueDataLen);
     if (lRet != ERROR_SUCCESS) {
-        error(hwnd, IDS_BAD_VALUE, valueName);
+        error_code_messagebox(hwnd, IDS_BAD_VALUE, valueName);
         goto done;
     }
     if((valueDataLen % sizeof(WCHAR)) == 0)
@@ -375,7 +370,7 @@ BOOL ModifyValue(HWND hwnd, HKEY hKeyRoot, LPCWSTR keyPath, LPCWSTR valueName)
             else error_code_messagebox(hwnd, IDS_SET_VALUE_FAILED);
         }
     } else {
-        error(hwnd, IDS_UNSUPPORTED_TYPE, type);
+        error_code_messagebox(hwnd, IDS_UNSUPPORTED_TYPE, type);
     }
 
 done:
@@ -402,7 +397,7 @@ BOOL DeleteKey(HWND hwnd, HKEY hKeyRoot, LPCWSTR keyPath)
 	
     lRet = SHDeleteKeyW(hKeyRoot, keyPath);
     if (lRet != ERROR_SUCCESS) {
-	error(hwnd, IDS_BAD_KEY, keyPath);
+        error_code_messagebox(hwnd, IDS_BAD_KEY, keyPath);
 	goto done;
     }
     result = TRUE;
@@ -431,7 +426,7 @@ BOOL DeleteValue(HWND hwnd, HKEY hKeyRoot, LPCWSTR keyPath, LPCWSTR valueName, B
 
     lRet = RegDeleteValueW(hKey, valueName ? valueName : &empty);
     if (lRet != ERROR_SUCCESS && valueName) {
-        error(hwnd, IDS_BAD_VALUE, valueName);
+        error_code_messagebox(hwnd, IDS_BAD_VALUE, valueName);
     }
     if (lRet != ERROR_SUCCESS) goto done;
     result = TRUE;
diff --git a/programs/regedit/framewnd.c b/programs/regedit/framewnd.c
index 43c581b..018759e 100644
--- a/programs/regedit/framewnd.c
+++ b/programs/regedit/framewnd.c
@@ -773,7 +773,7 @@ static BOOL _CmdWndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
                     SetFocus(g_pChildWnd->hTreeWnd);
                 }
             } else {
-                error(hWnd, IDS_NOTFOUND, searchString);
+                error_code_messagebox(hWnd, IDS_NOTFOUND, searchString);
             }
         }
         break;
diff --git a/programs/regedit/main.h b/programs/regedit/main.h
index 1528c41..bbf6c09 100644
--- a/programs/regedit/main.h
+++ b/programs/regedit/main.h
@@ -143,7 +143,7 @@ extern BOOL DeleteKey(HWND hwnd, HKEY hKeyRoot, LPCWSTR keyPath);
 extern BOOL DeleteValue(HWND hwnd, HKEY hKeyRoot, LPCWSTR keyPath, LPCWSTR valueName, BOOL showMessageBox);
 extern BOOL RenameValue(HWND hwnd, HKEY hRootKey, LPCWSTR keyPath, LPCWSTR oldName, LPCWSTR newName);
 extern BOOL RenameKey(HWND hwnd, HKEY hRootKey, LPCWSTR keyPath, LPCWSTR newName);
-extern void __cdecl error(HWND hwnd, INT resId, ...);
+extern void __cdecl error_code_messagebox(HWND hwnd, unsigned int msg_id, ...);
 
 /* hexedit.c */
 extern void HexEdit_Register(void);