[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