regedit: Use BOOL type where appropriate

Frédéric Delanoy frederic.delanoy at gmail.com
Wed Oct 16 04:12:20 CDT 2013


---
 programs/regedit/edit.c     | 14 +++++++-------
 programs/regedit/framewnd.c | 32 ++++++++++++++++----------------
 programs/regedit/listview.c |  6 +++---
 programs/regedit/treeview.c | 10 +++++-----
 4 files changed, 31 insertions(+), 31 deletions(-)

diff --git a/programs/regedit/edit.c b/programs/regedit/edit.c
index e211ff8..f9ed631 100644
--- a/programs/regedit/edit.c
+++ b/programs/regedit/edit.c
@@ -128,7 +128,7 @@ static INT_PTR CALLBACK modify_dlgproc(HWND hwndDlg, UINT uMsg, WPARAM wParam, L
         SetDlgItemTextW(hwndDlg, IDC_VALUE_NAME, editValueName);
         SetDlgItemTextW(hwndDlg, IDC_VALUE_DATA, stringValueData);
 	CheckRadioButton(hwndDlg, IDC_DWORD_HEX, IDC_DWORD_DEC, isDecimal ? IDC_DWORD_DEC : IDC_DWORD_HEX);
-        return TRUE;
+        return 1;
     case WM_COMMAND:
         switch (LOWORD(wParam)) {
         case IDC_DWORD_HEX:
@@ -149,10 +149,10 @@ static INT_PTR CALLBACK modify_dlgproc(HWND hwndDlg, UINT uMsg, WPARAM wParam, L
             /* Fall through */
         case IDCANCEL:
             EndDialog(hwndDlg, wParam);
-            return TRUE;
+            return 1;
         }
     }
-    return FALSE;
+    return 0;
 }
 
 static INT_PTR CALLBACK bin_modify_dlgproc(HWND hwndDlg, UINT uMsg, WPARAM wParam, LPARAM lParam)
@@ -171,7 +171,7 @@ static INT_PTR CALLBACK bin_modify_dlgproc(HWND hwndDlg, UINT uMsg, WPARAM wPara
         else
             SetDlgItemTextW(hwndDlg, IDC_VALUE_NAME, g_pszDefaultValueName);
         SendDlgItemMessageW(hwndDlg, IDC_VALUE_DATA, HEM_SETDATA, (WPARAM)params->cbData, (LPARAM)params->pData);
-        return TRUE;
+        return 1;
     case WM_COMMAND:
         switch (LOWORD(wParam)) {
         case IDOK:
@@ -194,13 +194,13 @@ static INT_PTR CALLBACK bin_modify_dlgproc(HWND hwndDlg, UINT uMsg, WPARAM wPara
                 error_code_messagebox(hwndDlg, lRet);
                 EndDialog(hwndDlg, 0);
             }
-            return TRUE;
+            return 1;
         case IDCANCEL:
             EndDialog(hwndDlg, 0);
-            return TRUE;
+            return 1;
         }
     }
-    return FALSE;
+    return 0;
 }
 
 static BOOL check_value(HWND hwnd, HKEY hKey, LPCWSTR valueName)
diff --git a/programs/regedit/framewnd.c b/programs/regedit/framewnd.c
index 2e28670..e56adcf 100644
--- a/programs/regedit/framewnd.c
+++ b/programs/regedit/framewnd.c
@@ -508,13 +508,13 @@ static INT_PTR CALLBACK find_dlgproc(HWND hwndDlg, UINT uMsg, WPARAM wParam, LPA
             CheckDlgButton(hwndDlg, IDC_FIND_WHOLE, searchMask&SEARCH_WHOLE ? BST_CHECKED : BST_UNCHECKED);
             SendMessageW(hwndValue, EM_SETLIMITTEXT, 127, 0);
             SetWindowTextW(hwndValue, searchString);
-            return TRUE;
+            return 1;
         case WM_COMMAND:
             switch(LOWORD(wParam)) {
             case IDC_VALUE_NAME:
                 if (HIWORD(wParam) == EN_UPDATE) {
                     EnableWindow(GetDlgItem(hwndDlg, IDOK),  GetWindowTextLengthW(hwndValue)>0);
-                    return TRUE;
+                    return 1;
                 }
                 break;
             case IDOK:
@@ -528,14 +528,14 @@ static INT_PTR CALLBACK find_dlgproc(HWND hwndDlg, UINT uMsg, WPARAM wParam, LPA
                     GetWindowTextW(hwndValue, searchString, 128);
                     EndDialog(hwndDlg, IDOK);
                 }
-                return TRUE;
+                return 1;
             case IDCANCEL:
                 EndDialog(hwndDlg, IDCANCEL);
-                return TRUE;
+                return 1;
             }
             break;
     }
-    return FALSE;
+    return 0;
 }
                     
 static INT_PTR CALLBACK addtofavorites_dlgproc(HWND hwndDlg, UINT uMsg, WPARAM wParam, LPARAM lParam)
@@ -554,14 +554,14 @@ static INT_PTR CALLBACK addtofavorites_dlgproc(HWND hwndDlg, UINT uMsg, WPARAM w
             SetWindowTextW(hwndValue, ItemPath);
             SendMessageW(hwndValue, EM_SETLIMITTEXT, 127, 0);
             HeapFree(GetProcessHeap(), 0, ItemPath);
-            return TRUE;
+            return 1;
         }
         case WM_COMMAND:
             switch(LOWORD(wParam)) {
             case IDC_VALUE_NAME:
                 if (HIWORD(wParam) == EN_UPDATE) {
                     EnableWindow(GetDlgItem(hwndDlg, IDOK), GetWindowTextLengthW(hwndValue) > 0);
-                    return TRUE;
+                    return 1;
                 }
                 break;
             case IDOK:
@@ -569,14 +569,14 @@ static INT_PTR CALLBACK addtofavorites_dlgproc(HWND hwndDlg, UINT uMsg, WPARAM w
                     GetWindowTextW(hwndValue, favoriteName, 128);
                     EndDialog(hwndDlg, IDOK);
                 }
-                return TRUE;
+                return 1;
             case IDCANCEL:
                 EndDialog(hwndDlg, IDCANCEL);
-                return TRUE;
+                return 1;
             }
             break;
     }
-    return FALSE;
+    return 0;
 }
                     
 static INT_PTR CALLBACK removefavorite_dlgproc(HWND hwndDlg, UINT uMsg, WPARAM wParam, LPARAM lParam)
@@ -608,7 +608,7 @@ static INT_PTR CALLBACK removefavorite_dlgproc(HWND hwndDlg, UINT uMsg, WPARAM w
                 RegCloseKey(hKey);
             }
             else
-                return FALSE;
+                return 0;
             EnableWindow(GetDlgItem(hwndDlg, IDOK), i != 0);
             SendMessageW(hwndList, LB_SETCURSEL, 0, 0);
             return TRUE;
@@ -618,7 +618,7 @@ static INT_PTR CALLBACK removefavorite_dlgproc(HWND hwndDlg, UINT uMsg, WPARAM w
             case IDC_NAME_LIST:
                 if (HIWORD(wParam) == LBN_SELCHANGE) {
                     EnableWindow(GetDlgItem(hwndDlg, IDOK),  lParam != -1);
-                    return TRUE;
+                    return 1;
                 }
                 break;
             case IDOK: {
@@ -633,17 +633,17 @@ static INT_PTR CALLBACK removefavorite_dlgproc(HWND hwndDlg, UINT uMsg, WPARAM w
                     EndDialog(hwndDlg, IDOK);
                     HeapFree(GetProcessHeap(), 0, lpName);
                 }
-                return TRUE;
+                return 1;
             }
             case IDCANCEL:
                 EndDialog(hwndDlg, IDCANCEL);
-                return TRUE;
+                return 1;
             }
             break;
     }
-    return FALSE;
+    return 0;
 }
-                    
+
 /*******************************************************************************
  *
  *  FUNCTION: _CmdWndProc(HWND, unsigned, WORD, LONG)
diff --git a/programs/regedit/listview.c b/programs/regedit/listview.c
index 6559116..b4dfd02 100644
--- a/programs/regedit/listview.c
+++ b/programs/regedit/listview.c
@@ -482,7 +482,7 @@ HWND CreateListView(HWND hwndParent, UINT id)
                             0, 0, rcClient.right, rcClient.bottom,
                             hwndParent, ULongToHandle(id), hInst, NULL);
     if (!hwndLV) return NULL;
-    SendMessageW(hwndLV, LVM_SETUNICODEFORMAT, TRUE, 0);
+    SendMessageW(hwndLV, LVM_SETUNICODEFORMAT, 1, 0);
     SendMessageW(hwndLV, LVM_SETEXTENDEDLISTVIEWSTYLE, 0, LVS_EX_FULLROWSELECT);
 
     /* Initialize the image list */
@@ -511,7 +511,7 @@ BOOL RefreshListView(HWND hwndLV, HKEY hKeyRoot, LPCWSTR keyPath, LPCWSTR highli
 
     if (!hwndLV) return FALSE;
 
-    SendMessageW(hwndLV, WM_SETREDRAW, FALSE, 0);
+    SendMessageW(hwndLV, WM_SETREDRAW, 0, 0);
 
     errCode = RegOpenKeyExW(hKeyRoot, keyPath, 0, KEY_READ, &hKey);
     if (errCode != ERROR_SUCCESS) goto done;
@@ -573,7 +573,7 @@ BOOL RefreshListView(HWND hwndLV, HKEY hKeyRoot, LPCWSTR keyPath, LPCWSTR highli
 done:
     HeapFree(GetProcessHeap(), 0, valBuf);
     HeapFree(GetProcessHeap(), 0, valName);
-    SendMessageW(hwndLV, WM_SETREDRAW, TRUE, 0);
+    SendMessageW(hwndLV, WM_SETREDRAW, 1, 0);
     if (hKey) RegCloseKey(hKey);
 
     return result;
diff --git a/programs/regedit/treeview.c b/programs/regedit/treeview.c
index 97a0bf2..5e36026 100644
--- a/programs/regedit/treeview.c
+++ b/programs/regedit/treeview.c
@@ -457,7 +457,7 @@ BOOL RefreshTreeView(HWND hwndTV)
     WINE_TRACE("\n");
     hSelectedItem = (HTREEITEM)SendMessageW(hwndTV, TVM_GETNEXTITEM, TVGN_CARET, 0);
     hcursorOld = SetCursor(LoadCursorW(NULL, (LPCWSTR)IDC_WAIT));
-    SendMessageW(hwndTV, WM_SETREDRAW, FALSE, 0);
+    SendMessageW(hwndTV, WM_SETREDRAW, 0, 0);
 
     hRoot = (HTREEITEM)SendMessageW(hwndTV, TVM_GETNEXTITEM, TVGN_ROOT, 0);
     hItem = (HTREEITEM)SendMessageW(hwndTV, TVM_GETNEXTITEM, TVGN_CHILD, (LPARAM)hRoot);
@@ -466,7 +466,7 @@ BOOL RefreshTreeView(HWND hwndTV)
         hItem = (HTREEITEM)SendMessageW(hwndTV, TVM_GETNEXTITEM, TVGN_NEXT, (LPARAM)hItem);
     }
 
-    SendMessageW(hwndTV, WM_SETREDRAW, TRUE, 0);
+    SendMessageW(hwndTV, WM_SETREDRAW, 1, 0);
     InvalidateRect(hwndTV, NULL, FALSE);
     SetCursor(hcursorOld);
     
@@ -613,7 +613,7 @@ BOOL UpdateExpandingTree(HWND hwndTV, HTREEITEM hItem, int state)
     }
     expanding = TRUE;
     hcursorOld = SetCursor(LoadCursorW(NULL, (LPCWSTR)IDC_WAIT));
-    SendMessageW(hwndTV, WM_SETREDRAW, FALSE, 0);
+    SendMessageW(hwndTV, WM_SETREDRAW, 0, 0);
 
     keyPath = GetItemPath(hwndTV, hItem, &hRoot);
     if (!keyPath) goto done;
@@ -653,7 +653,7 @@ done:
     item.stateMask = TVIS_EXPANDEDONCE;
     item.state = TVIS_EXPANDEDONCE;
     SendMessageW(hwndTV, TVM_SETITEMW, 0, (LPARAM)&item);
-    SendMessageW(hwndTV, WM_SETREDRAW, TRUE, 0);
+    SendMessageW(hwndTV, WM_SETREDRAW, 1, 0);
     SetCursor(hcursorOld);
     expanding = FALSE;
     HeapFree(GetProcessHeap(), 0, keyPath);
@@ -684,7 +684,7 @@ HWND CreateTreeView(HWND hwndParent, LPWSTR pHostName, UINT id)
                             WS_VISIBLE | WS_CHILD | WS_TABSTOP | TVS_HASLINES | TVS_HASBUTTONS | TVS_LINESATROOT | TVS_EDITLABELS,
                             0, 0, rcClient.right, rcClient.bottom,
                             hwndParent, ULongToHandle(id), hInst, NULL);
-    SendMessageW(hwndTV, TVM_SETUNICODEFORMAT, TRUE, 0);
+    SendMessageW(hwndTV, TVM_SETUNICODEFORMAT, 1, 0);
     /* Initialize the image list, and add items to the control.  */
     if (!InitTreeViewImageLists(hwndTV) || !InitTreeViewItems(hwndTV, pHostName)) {
         DestroyWindow(hwndTV);
-- 
1.8.4




More information about the wine-patches mailing list