Re: [PATCH v3 0/2] MR5634: win32u: Ignore WM_MOUSEMOVE when tracking scrollbar and arrow/track is clicked

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/win32u/scroll.c:
> if (msg == WM_LBUTTONDOWN || msg == WM_SYSTIMER)
>          {
>              send_message( owner_hwnd, vertical ? WM_VSCROLL : WM_HSCROLL, msg_send, (LPARAM)ctl_hwnd );
> +            scroll_timer_running = FALSE;
>          }
>
> +        if (!scroll_timer_running)
> +        {

```suggestion:-4+0
        }

        if (!scroll_timer_running || msg == WM_LBUTTONDOWN || msg == WM_SYSTIMER)
        {
```

I'd do that instead, to avoid overriding the scroll_timer_running semantic.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5634#note_70676