Michael Stefaniuc : krnl386.exe16: Avoid ARRAY_SIZE-like macros.

[Alexandre Julliard]

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

Author: Michael Stefaniuc 
Date:   Thu Jul  5 22:07:23 2018 +0200

krnl386.exe16: Avoid ARRAY_SIZE-like macros.

Signed-off-by: Michael Stefaniuc 
Signed-off-by: Alexandre Julliard

---

dlls/krnl386.exe16/error.c | 6 ++----
 dlls/krnl386.exe16/int21.c | 8 +++-----
 dlls/krnl386.exe16/vxd.c   | 4 +---
 3 files changed, 6 insertions(+), 12 deletions(-)

diff --git a/dlls/krnl386.exe16/error.c b/dlls/krnl386.exe16/error.c
index dc02d21..5508d7c 100644
--- a/dlls/krnl386.exe16/error.c
+++ b/dlls/krnl386.exe16/error.c
@@ -193,8 +193,6 @@ static const struct {
 };

#undef  ErrorString
-#define ErrorStringCount (sizeof(ErrorStrings) / sizeof(ErrorStrings[0]))
-#define ParamErrorStringCount (sizeof(ParamErrorStrings) / sizeof(ParamErrorStrings[0]))

/***********************************************************************
 *	GetErrorString (internal)
@@ -204,7 +202,7 @@ static const char *GetErrorString(UINT16 uErr)
   static char buffer[80];
   unsigned int n;

-  for (n = 0; n < ErrorStringCount; n++) {
+  for (n = 0; n < ARRAY_SIZE(ErrorStrings); n++) {
     if (uErr == ErrorStrings[n].constant)
       return ErrorStrings[n].name;
   }
@@ -229,7 +227,7 @@ static const char *GetParamErrorString(UINT16 uErr) {
    {
    	unsigned int n;

-		for (n = 0; n < ParamErrorStringCount; n++) {
+		for (n = 0; n < ARRAY_SIZE(ParamErrorStrings); n++) {
    		if (uErr == ParamErrorStrings[n].constant) {
    			strcat(buffer, ParamErrorStrings[n].name);
    			return buffer;
diff --git a/dlls/krnl386.exe16/int21.c b/dlls/krnl386.exe16/int21.c
index 3a08749..f32b1a7 100644
--- a/dlls/krnl386.exe16/int21.c
+++ b/dlls/krnl386.exe16/int21.c
@@ -282,8 +282,6 @@ static struct magic_device magic_devices[] =
     { {'h','p','s','c','a','n',0},         NULL, { { 0, 0 } }, INT21_IoctlHPScanHandler },
 };

-#define NB_MAGIC_DEVICES  (sizeof(magic_devices)/sizeof(magic_devices[0]))
-

/* Many calls translate a drive argument like this:
    drive number (00h = default, 01h = A:, etc)
@@ -856,13 +854,13 @@ static HANDLE INT21_OpenMagicDevice( LPCWSTR name, DWORD access )
     if ((p = strrchrW( name, '/' ))) name = p + 1;
     if ((p = strrchrW( name, '\' ))) name = p + 1;

-    for (i = 0; i < NB_MAGIC_DEVICES; i++)
+    for (i = 0; i < ARRAY_SIZE(magic_devices); i++)
     {
         int len = strlenW( magic_devices[i].name );
         if (!strncmpiW( magic_devices[i].name, name, len ) &&
             (!name[len] || name[len] == '.' || name[len] == ':')) break;
     }
-    if (i == NB_MAGIC_DEVICES) return 0;
+    if (i == ARRAY_SIZE(magic_devices)) return 0;

if (!magic_devices[i].handle) /* need to open it */
     {
@@ -2639,7 +2637,7 @@ static void INT21_Ioctl_Char( CONTEXT *context )
         }
     } else {
         UINT i;
-        for (i = 0; i < NB_MAGIC_DEVICES; i++)
+        for (i = 0; i < ARRAY_SIZE(magic_devices); i++)
         {
             if (!magic_devices[i].handle) continue;
             if (magic_devices[i].index.QuadPart == info.IndexNumber.QuadPart)
diff --git a/dlls/krnl386.exe16/vxd.c b/dlls/krnl386.exe16/vxd.c
index 7be5f63..dc14600f 100644
--- a/dlls/krnl386.exe16/vxd.c
+++ b/dlls/krnl386.exe16/vxd.c
@@ -80,8 +80,6 @@ static struct vxdcall_service vxd_services[] =
     { {'v','w','i','n','3','2','.','v','x','d',0}, 0x002a, NULL, NULL }
 };

-#define NB_VXD_SERVICES  (sizeof(vxd_services)/sizeof(vxd_services[0]))
-
 #define W32S_APP2WINE(addr) ((addr)? (DWORD)(addr) + W32S_offset : 0)
 #define W32S_WINE2APP(addr) ((addr)? (DWORD)(addr) - W32S_offset : 0)

@@ -293,7 +291,7 @@ void WINAPI DECLSPEC_HIDDEN __regs_VxDCall( CONTEXT *context )
     DWORD service = stack32_pop( context );

RtlEnterCriticalSection( &vxd_section );
-    for (i = 0; i < NB_VXD_SERVICES; i++)
+    for (i = 0; i < ARRAY_SIZE(vxd_services); i++)
     {
         if (HIWORD(service) != vxd_services[i].service) continue;
         if (!vxd_services[i].module)  /* need to load it */