Michael Stefaniuc : winecfg: Use the ARRAY_SIZE() macro.

Alexandre Julliard julliard at winehq.org
Mon Oct 29 18:26:30 CDT 2018


Module: wine
Branch: master
Commit: f051aa6774194689b1ab6aaba3b9c22be902d6ab
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=f051aa6774194689b1ab6aaba3b9c22be902d6ab

Author: Michael Stefaniuc <mstefani at winehq.org>
Date:   Mon Oct 29 20:10:42 2018 +0100

winecfg: Use the ARRAY_SIZE() macro.

Signed-off-by: Michael Stefaniuc <mstefani at winehq.org>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 programs/winecfg/driveui.c |  2 +-
 programs/winecfg/theme.c   | 12 +++++-------
 programs/winecfg/winecfg.c |  2 +-
 3 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/programs/winecfg/driveui.c b/programs/winecfg/driveui.c
index c25e71d..36ae88c 100644
--- a/programs/winecfg/driveui.c
+++ b/programs/winecfg/driveui.c
@@ -440,7 +440,7 @@ static void update_controls(HWND dialog)
     type = current_drive->type;
     SendDlgItemMessageW(dialog, IDC_COMBO_TYPE, CB_RESETCONTENT, 0, 0);
 
-    for (i = 0; i < sizeof(type_pairs) / sizeof(struct drive_typemap); i++)
+    for (i = 0; i < ARRAY_SIZE(type_pairs); i++)
     {
         WCHAR driveDesc[64];
         LoadStringW(GetModuleHandleW(NULL), type_pairs[i].idDesc, driveDesc, ARRAY_SIZE(driveDesc));
diff --git a/programs/winecfg/theme.c b/programs/winecfg/theme.c
index 5304f72..78d89a5 100644
--- a/programs/winecfg/theme.c
+++ b/programs/winecfg/theme.c
@@ -635,7 +635,7 @@ static void on_theme_install(HWND dialog)
   ofn.nFilterIndex = 0;
   ofn.lpstrFile = file;
   ofn.lpstrFile[0] = '\0';
-  ofn.nMaxFile = sizeof(file)/sizeof(filetitle[0]);
+  ofn.nMaxFile = ARRAY_SIZE(file);
   ofn.lpstrFileTitle = filetitle;
   ofn.lpstrFileTitle[0] = '\0';
   ofn.nMaxFileTitle = ARRAY_SIZE(filetitle);
@@ -729,8 +729,6 @@ static struct ShellFolderInfo asfiInfo[] = {
 
 static struct ShellFolderInfo *psfiSelected = NULL;
 
-#define NUM_ELEMS(x) (sizeof(x)/sizeof(*(x)))
-
 static void init_shell_folder_listview_headers(HWND dialog) {
     LVCOLUMNW listColumn;
     RECT viewRect;
@@ -763,8 +761,8 @@ static void read_shell_folder_link_targets(void) {
     WCHAR wszPath[MAX_PATH];
     HRESULT hr;
     int i;
-   
-    for (i=0; i<NUM_ELEMS(asfiInfo); i++) {
+
+    for (i=0; i<ARRAY_SIZE(asfiInfo); i++) {
         asfiInfo[i].szLinkTarget[0] = '\0';
         hr = SHGetFolderPathW(NULL, asfiInfo[i].nFolder|CSIDL_FLAG_DONT_VERIFY, NULL, 
                               SHGFP_TYPE_CURRENT, wszPath);
@@ -790,7 +788,7 @@ static void update_shell_folder_listview(HWND dialog) {
 
     SendDlgItemMessageW(dialog, IDC_LIST_SFPATHS, LVM_DELETEALLITEMS, 0, 0);
 
-    for (i=0; i<NUM_ELEMS(asfiInfo); i++) {
+    for (i=0; i<ARRAY_SIZE(asfiInfo); i++) {
         WCHAR buffer[MAX_PATH];
         HRESULT hr;
         LPITEMIDLIST pidlCurrent;
@@ -905,7 +903,7 @@ static void apply_shell_folder_changes(void) {
     struct stat statPath;
     HRESULT hr;
 
-    for (i=0; i<NUM_ELEMS(asfiInfo); i++) {
+    for (i=0; i<ARRAY_SIZE(asfiInfo); i++) {
         /* Ignore nonexistent link targets */
         if (asfiInfo[i].szLinkTarget[0] && stat(asfiInfo[i].szLinkTarget, &statPath))
             continue;
diff --git a/programs/winecfg/winecfg.c b/programs/winecfg/winecfg.c
index be04ba2..c4c05bd 100644
--- a/programs/winecfg/winecfg.c
+++ b/programs/winecfg/winecfg.c
@@ -623,7 +623,7 @@ static void process_setting(struct setting *s)
     static const WCHAR softwareW[] = {'S','o','f','t','w','a','r','e','\\'};
     HKEY key;
     BOOL needs_wow64 = (is_win64 && s->root == HKEY_LOCAL_MACHINE && s->path &&
-                        !strncmpiW( s->path, softwareW, sizeof(softwareW)/sizeof(WCHAR) ));
+                        !strncmpiW(s->path, softwareW, ARRAY_SIZE(softwareW)));
 
     if (s->value)
     {




More information about the wine-cvs mailing list