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

[Rémi Bernon]

On 5/14/21 9:15 AM, Rémi Bernon wrote:
> Signed-off-by: Rémi Bernon
>
> 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.