[PATCH 0/4] MR233: win32u: Partially move raw input APIs.

[Rémi Bernon (@rbernon)]

Rémi Bernon (@rbernon) commented about dlls/user32/tests/input.c:
>      ok(GetLastError() == ERROR_INVALID_PARAMETER, "GetRawInputBuffer returned %08lx\n", GetLastError());
>  
> +    SetLastError(0xdeadbeef);
> +    size = sizeof(buffer);
> +    count = GetRawInputBuffer((RAWINPUT*)buffer, &size, sizeof(RAWINPUTHEADER) + 1);
> +    ok(count == ~0U, "GetRawInputBuffer succeeded\n");
> +    ok(GetLastError() == ERROR_INVALID_PARAMETER, "GetRawInputBuffer returned %08lx\n", GetLastError());
> +
> +    /* the function returns 64-bit RAWINPUT structures on WoW64, but still
> +     * forbids sizeof(RAWINPUTHEADER) from the wrong architecture */
> +    SetLastError(0xdeadbeef);
> +    size = sizeof(buffer);
> +    if (sizeof(void *) == 8)
> +        count = GetRawInputBuffer((RAWINPUT*)buffer, &size, sizeof(RAWINPUTHEADER32));
> +    else
> +        count = GetRawInputBuffer((RAWINPUT*)buffer, &size, sizeof(RAWINPUTHEADER64));
Same here, I think adding a conditionally initialized `header_size` variable would make it simpler to read.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/233#note_1937