Re: [PATCH 0/1] MR7736: server: Fix the accumulation method when merging WM_MOUSEWHEEL message.

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about server/queue.c:
> static int merge_mousewheel( struct thread_input *input, const struct message *msg )
>  {
>      struct message *prev;
> +    short delta;
>
> if (!(prev = find_mouse_message( input, msg ))) return 0;
>      if (prev->x != msg->x || prev->y != msg->y) return 0; /* don't merge if cursor has moved */
>
> -    prev->wparam += msg->wparam; /* accumulate wheel delta */
> +    /* accumulate wheel delta */
> +    delta = GET_WHEEL_DELTA_WPARAM(prev->wparam) + GET_WHEEL_DELTA_WPARAM(msg->wparam);
> Should we clamp it, or avoid merging on overflow maybe?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7736#note_99859