[PATCH v2 1/4] user32/tests: Count WM_INPUT messages only as rawinputbuffer iterations.

Rémi Bernon rbernon at codeweavers.com
Fri May 14 04:49:04 CDT 2021


On 5/14/21 9:15 AM, Rémi Bernon wrote:
> Signed-off-by: Rémi Bernon <rbernon at codeweavers.com>
> ---
> 
> This is likely to be the cause of some of the spurious failures.
> 
> v2: (PATCH 3) Check hid.length vs request data size and prefer request
>      data size.
> 
>   dlls/user32/tests/input.c | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/dlls/user32/tests/input.c b/dlls/user32/tests/input.c
> index 3d4f0d03d0d..d7e7c09650f 100644
> --- a/dlls/user32/tests/input.c
> +++ b/dlls/user32/tests/input.c
> @@ -2138,7 +2138,7 @@ static LRESULT CALLBACK rawinputbuffer_wndproc(HWND hwnd, UINT msg, WPARAM wpara
>   {
>       RAWINPUT ri;
>       char buffer[16 * sizeof(RAWINPUT64)];
> -    UINT size, count, rawinput_size, iteration = rawinputbuffer_wndproc_count++;
> +    UINT size, count, rawinput_size, iteration;
>       MSG message;
>   
>       if (is_wow64) rawinput_size = sizeof(RAWINPUT64);
> @@ -2146,6 +2146,8 @@ static LRESULT CALLBACK rawinputbuffer_wndproc(HWND hwnd, UINT msg, WPARAM wpara
>   
>       if (msg == WM_INPUT)
>       {
> +        iteration = rawinputbuffer_wndproc_count++;
> +
>           count = GetRawInputBuffer(NULL, NULL, sizeof(RAWINPUTHEADER));
>           ok(count == ~0U, "GetRawInputBuffer succeeded\n");
>   
> 

This first patch can simply be ignored, I've sent some updated version 
to try to fix the failures (although it's hard to reproduce).

I think the other three patches are pretty much unrelated to the issue.
-- 
Rémi Bernon <rbernon at codeweavers.com>



More information about the wine-devel mailing list