Michael Stefaniuc : sane.ds: Avoid an ARRAY_SIZE-like macro.

Alexandre Julliard julliard at winehq.org
Thu Aug 16 13:42:51 CDT 2018


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

Author: Michael Stefaniuc <mstefani at winehq.org>
Date:   Thu Aug 16 20:19:25 2018 +0200

sane.ds: Avoid an ARRAY_SIZE-like macro.

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

---

 dlls/sane.ds/capability.c | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/dlls/sane.ds/capability.c b/dlls/sane.ds/capability.c
index 45c54a1..094bb93 100644
--- a/dlls/sane.ds/capability.c
+++ b/dlls/sane.ds/capability.c
@@ -844,7 +844,6 @@ static const supported_size_t supported_sizes[] =
     { TWSS_USEXECUTIVE, 184.15, 266.7   },
     { TWSS_A3,          297,    420     },
 };
-#define SUPPORTED_SIZE_COUNT (sizeof(supported_sizes) / sizeof(supported_sizes[0]))
 
 static TW_UINT16 get_default_paper_size(const supported_size_t *s, int n)
 {
@@ -920,11 +919,11 @@ static TW_UINT16 SANE_ICAPSupportedSizes (pTW_CAPABILITY pCapability, TW_UINT16
     TW_UINT16 twCC = TWCC_BADCAP;
 #ifdef SONAME_LIBSANE
 
-    static TW_UINT32 possible_values[SUPPORTED_SIZE_COUNT];
+    static TW_UINT32 possible_values[ARRAY_SIZE(supported_sizes)];
     unsigned int i;
     TW_UINT32 val;
-    TW_UINT16 default_size = get_default_paper_size(supported_sizes, SUPPORTED_SIZE_COUNT);
-    TW_UINT16 current_size = get_current_paper_size(supported_sizes, SUPPORTED_SIZE_COUNT);
+    TW_UINT16 default_size = get_default_paper_size(supported_sizes, ARRAY_SIZE(supported_sizes));
+    TW_UINT16 current_size = get_current_paper_size(supported_sizes, ARRAY_SIZE(supported_sizes));
 
     TRACE("ICAP_SUPPORTEDSIZES\n");
 
@@ -946,7 +945,7 @@ static TW_UINT16 SANE_ICAPSupportedSizes (pTW_CAPABILITY pCapability, TW_UINT16
         case MSG_SET:
             twCC = msg_set(pCapability, &val);
             if (twCC == TWCC_SUCCESS)
-                for (i = 1; i < SUPPORTED_SIZE_COUNT; i++)
+                for (i = 1; i < ARRAY_SIZE(supported_sizes); i++)
                     if (supported_sizes[i].size == val)
                         return set_width_height(supported_sizes[i].x, supported_sizes[i].y);
 
@@ -960,7 +959,7 @@ static TW_UINT16 SANE_ICAPSupportedSizes (pTW_CAPABILITY pCapability, TW_UINT16
 
         case MSG_RESET:
             twCC = TWCC_BADCAP;
-            for (i = 1; i < SUPPORTED_SIZE_COUNT; i++)
+            for (i = 1; i < ARRAY_SIZE(supported_sizes); i++)
                 if (supported_sizes[i].size == default_size)
                 {
                     twCC = set_width_height(supported_sizes[i].x, supported_sizes[i].y);
@@ -976,7 +975,6 @@ static TW_UINT16 SANE_ICAPSupportedSizes (pTW_CAPABILITY pCapability, TW_UINT16
             break;
     }
 
-#undef SUPPORTED_SIZE_COUNT
 #endif
     return twCC;
 }




More information about the wine-cvs mailing list