Re: [PATCH 0/1] MR10830: Draft: winewayland: Use wp_pointer_warp_v1 for SetCursorPos when available.

[Etaash Mathamsetty (＠etaash.mathamsetty)]

Etaash Mathamsetty (@etaash.mathamsetty) commented about dlls/winewayland.drv/wayland_pointer.c:
> pthread_mutex_lock(&pointer->mutex);
>      if (wl_surface && pointer->pending_warp)
>      {
> -        wayland_pointer_update_constraint(wl_surface, NULL, FALSE, TRUE);
> +        if (pointer->enter_serial && process_wayland.wp_pointer_warp_v1)

The implementation looks correct, but I don't think acquiring the pointer lock does anything unless the pointer is within the window, so the logic can be simplified by not checking for this. But that's a rather minor nitpick

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10830#note_139181