Commit 0dde234e authored by Michael Stefaniuc's avatar Michael Stefaniuc Committed by Alexandre Julliard

imm32/tests: Use the available ARRAY_SIZE() macro.

parent 05253c75
...@@ -26,8 +26,6 @@ ...@@ -26,8 +26,6 @@
#include "imm.h" #include "imm.h"
#include "ddk/imm.h" #include "ddk/imm.h"
#define NUMELEMS(array) (sizeof((array))/sizeof((array)[0]))
static BOOL (WINAPI *pImmAssociateContextEx)(HWND,HIMC,DWORD); static BOOL (WINAPI *pImmAssociateContextEx)(HWND,HIMC,DWORD);
static BOOL (WINAPI *pImmIsUIMessageA)(HWND,UINT,WPARAM,LPARAM); static BOOL (WINAPI *pImmIsUIMessageA)(HWND,UINT,WPARAM,LPARAM);
static UINT (WINAPI *pSendInput) (UINT, INPUT*, size_t); static UINT (WINAPI *pSendInput) (UINT, INPUT*, size_t);
...@@ -73,7 +71,7 @@ static LRESULT CALLBACK get_msg_filter(int nCode, WPARAM wParam, LPARAM lParam) ...@@ -73,7 +71,7 @@ static LRESULT CALLBACK get_msg_filter(int nCode, WPARAM wParam, LPARAM lParam)
MSG *msg = (MSG*)lParam; MSG *msg = (MSG*)lParam;
if ((msg->hwnd == msg_spy.hwnd || msg_spy.hwnd == NULL) && 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.hwnd = msg->hwnd;
msg_spy.msgs[msg_spy.i_msg].msg.message = msg->message; 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, ...@@ -94,7 +92,7 @@ static LRESULT CALLBACK call_wnd_proc_filter(int nCode, WPARAM wParam,
CWPSTRUCT *cwp = (CWPSTRUCT*)lParam; CWPSTRUCT *cwp = (CWPSTRUCT*)lParam;
if (((cwp->hwnd == msg_spy.hwnd || msg_spy.hwnd == NULL)) && 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)); 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; 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) { ...@@ -126,7 +124,7 @@ static imm_msgs* msg_spy_find_next_msg(UINT message, UINT *start) {
msg_spy_pump_msg_queue(); 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", fprintf(stdout, "%s:%d: msg_spy: message buffer overflow!\n",
__FILE__, __LINE__); __FILE__, __LINE__);
...@@ -1067,8 +1065,8 @@ static BOOL CALLBACK is_ime_window_proc(HWND hWnd, LPARAM param) ...@@ -1067,8 +1065,8 @@ static BOOL CALLBACK is_ime_window_proc(HWND hWnd, LPARAM param)
static const WCHAR imeW[] = {'I','M','E',0}; static const WCHAR imeW[] = {'I','M','E',0};
WCHAR class_nameW[16]; WCHAR class_nameW[16];
HWND *ime_window = (HWND *)param; HWND *ime_window = (HWND *)param;
if (GetClassNameW(hWnd, class_nameW, sizeof(class_nameW)/sizeof(class_nameW[0])) && if (GetClassNameW(hWnd, class_nameW, ARRAY_SIZE(class_nameW)) && !lstrcmpW(class_nameW, imeW))
!lstrcmpW(class_nameW, imeW)) { {
*ime_window = hWnd; *ime_window = hWnd;
return FALSE; return FALSE;
} }
...@@ -1229,7 +1227,7 @@ static void test_default_ime_window_creation(void) ...@@ -1229,7 +1227,7 @@ static void test_default_ime_window_creation(void)
{ FALSE, FALSE } { 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); thread = CreateThread(NULL, 0, test_default_ime_window_cb, &testcases[i], 0, NULL);
ok(thread != NULL, "CreateThread failed with error %u\n", GetLastError()); ok(thread != NULL, "CreateThread failed with error %u\n", GetLastError());
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment