On Fri Jun 16 17:20:24 2023 +0000, Rémi Bernon wrote:
Maybe it would be better to use `NtUserClipCursor( NULL )` here, I don't think ungrabbing the cursor from another process does anything, and I'd say it's not completely obvious that the current thread is also the foreground one? Maybe test whether WM_SYSCOMMAND does that already? Is the cursor still constrained on Windows when you move the window? If not, maybe we can just clear the Win32 clipping rect regardless?
To be honest, it seems only the first and last patches are needed now (it didn't use to be the case, maybe things changed). So you're right that maybe SYSCOMMAND does that already now.
Note that without the last patch in win32u, you can't drag windows on top of a fullscreen window at all.
I will remove these 2 patches then and only keep the bare necessary. Thanks for the review!