[PATCH] imm32/tests: Use the available ARRAY_SIZE() macro
Aric Stewart
aric at codeweavers.com
Wed Jun 6 10:19:42 CDT 2018
Signed-off-by: Aric Stewart <aric at codeweavers.com>
On 6/5/18 1:09 PM, Michael Stefaniuc wrote:
> Signed-off-by: Michael Stefaniuc <mstefani at winehq.org>
> ---
> dlls/imm32/tests/imm32.c | 14 ++++++--------
> 1 file changed, 6 insertions(+), 8 deletions(-)
>
> diff --git a/dlls/imm32/tests/imm32.c b/dlls/imm32/tests/imm32.c
> index be3ddfce03..ee1aeb3965 100644
> --- a/dlls/imm32/tests/imm32.c
> +++ b/dlls/imm32/tests/imm32.c
> @@ -26,8 +26,6 @@
> #include "imm.h"
> #include "ddk/imm.h"
>
> -#define NUMELEMS(array) (sizeof((array))/sizeof((array)[0]))
> -
> static BOOL (WINAPI *pImmAssociateContextEx)(HWND,HIMC,DWORD);
> static BOOL (WINAPI *pImmIsUIMessageA)(HWND,UINT,WPARAM,LPARAM);
> static UINT (WINAPI *pSendInput) (UINT, INPUT*, size_t);
> @@ -73,7 +71,7 @@ static LRESULT CALLBACK get_msg_filter(int nCode, WPARAM wParam, LPARAM lParam)
> MSG *msg = (MSG*)lParam;
>
> if ((msg->hwnd == msg_spy.hwnd || msg_spy.hwnd == NULL) &&
> - (msg_spy.i_msg < NUMELEMS(msg_spy.msgs)))
> + (msg_spy.i_msg < ARRAY_SIZE(msg_spy.msgs)))
> {
> msg_spy.msgs[msg_spy.i_msg].msg.hwnd = msg->hwnd;
> msg_spy.msgs[msg_spy.i_msg].msg.message = msg->message;
> @@ -94,7 +92,7 @@ static LRESULT CALLBACK call_wnd_proc_filter(int nCode, WPARAM wParam,
> CWPSTRUCT *cwp = (CWPSTRUCT*)lParam;
>
> if (((cwp->hwnd == msg_spy.hwnd || msg_spy.hwnd == NULL)) &&
> - (msg_spy.i_msg < NUMELEMS(msg_spy.msgs)))
> + (msg_spy.i_msg < ARRAY_SIZE(msg_spy.msgs)))
> {
> memcpy(&msg_spy.msgs[msg_spy.i_msg].msg, cwp, sizeof(msg_spy.msgs[0].msg));
> msg_spy.msgs[msg_spy.i_msg].post = FALSE;
> @@ -126,7 +124,7 @@ static imm_msgs* msg_spy_find_next_msg(UINT message, UINT *start) {
>
> msg_spy_pump_msg_queue();
>
> - if (msg_spy.i_msg >= NUMELEMS(msg_spy.msgs))
> + if (msg_spy.i_msg >= ARRAY_SIZE(msg_spy.msgs))
> fprintf(stdout, "%s:%d: msg_spy: message buffer overflow!\n",
> __FILE__, __LINE__);
>
> @@ -1067,8 +1065,8 @@ static BOOL CALLBACK is_ime_window_proc(HWND hWnd, LPARAM param)
> static const WCHAR imeW[] = {'I','M','E',0};
> WCHAR class_nameW[16];
> HWND *ime_window = (HWND *)param;
> - if (GetClassNameW(hWnd, class_nameW, sizeof(class_nameW)/sizeof(class_nameW[0])) &&
> - !lstrcmpW(class_nameW, imeW)) {
> + if (GetClassNameW(hWnd, class_nameW, ARRAY_SIZE(class_nameW)) && !lstrcmpW(class_nameW, imeW))
> + {
> *ime_window = hWnd;
> return FALSE;
> }
> @@ -1229,7 +1227,7 @@ static void test_default_ime_window_creation(void)
> { FALSE, FALSE }
> };
>
> - for (i = 0; i < sizeof(testcases)/sizeof(testcases[0]); i++)
> + for (i = 0; i < ARRAY_SIZE(testcases); i++)
> {
> thread = CreateThread(NULL, 0, test_default_ime_window_cb, &testcases[i], 0, NULL);
> ok(thread != NULL, "CreateThread failed with error %u\n", GetLastError());
>
More information about the wine-devel
mailing list