[PATCH v2 3/4] server: Add HID length and report to rawinput union.

Alexandre Julliard julliard at winehq.org
Fri May 14 14:38:46 CDT 2021


Rémi Bernon <rbernon at codeweavers.com> writes:

> @@ -302,6 +302,9 @@ union rawinput
>          unsigned int   param;   /* rawinput message param */
>          unsigned short usage_page;/* HID usage page */
>          unsigned short usage;   /* HID usage */
> +        unsigned int   length;  /* HID report length */
> +        int            __pad;
> +        /* followed by length bytes of HID report data  */

That's not true in server requests, I don't think we should pretend that
it is. The report data should be declared as a separate VARARG field in
the request, and use a separate pointer where necessary. The length can
probably be stored separately as well.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list