[2/2] user32: do not call NULL message callback
Robert Shearman
rob at codeweavers.com
Sun Feb 25 07:28:49 CST 2007
Fabian Bieler wrote:
> @@ -8894,23 +8894,20 @@ static void test_nullCallback(void)
> HWND hwnd;
> MSG msg;
>
> - if(0)
> - {
> - hwnd = CreateWindowExA(0, "TestWindowClass", "Test overlapped", WS_OVERLAPPEDWINDOW,
> - 100, 100, 200, 200, 0, 0, 0, NULL);
> - ok (hwnd != 0, "Failed to create overlapped window\n");
> + hwnd = CreateWindowExA(0, "TestWindowClass", "Test overlapped", WS_OVERLAPPEDWINDOW,
> + 100, 100, 200, 200, 0, 0, 0, NULL);
> + ok (hwnd != 0, "Failed to create overlapped window\n");
>
> - ShowWindow(hwnd, SW_SHOW);
> - UpdateWindow( hwnd );
> + ShowWindow(hwnd, SW_SHOW);
> + UpdateWindow( hwnd );
>
> - SendMessageCallbackA(hwnd,WM_NULL,0,0,NULL,0);
> + SendMessageCallbackA(hwnd,WM_NULL,0,0,NULL,0);
>
> - while (PeekMessage( &msg, 0, 0, 0, PM_REMOVE )) DispatchMessage( &msg );
> + while (PeekMessage( &msg, 0, 0, 0, PM_REMOVE )) DispatchMessage( &msg );
>
> - ok(1, "\n");
> + ok(1, "\n");
>
> - DestroyWindow(hwnd);
> - }
> + DestroyWindow(hwnd);
> }
>
> START_TEST(msg)
>
You might as well add the test and fix the bug in one patch since the
second patch is no shorter due to all of the whitespace changes.
--
Rob Shearman
More information about the wine-devel
mailing list