[PATCH] winex11.drv/window: Sync the window cursor before grabbing it

Gabriel Ivăncescu gabrielopcode at gmail.com
Wed Dec 19 05:23:26 CST 2018


On 12/18/18 22:52, Marvin wrote:
> Thank you for your contribution to Wine!
>
> This is an automated notification to let you know that your patch has
> been reviewed and its status set to "Deferred".
>
> This means that the patch is deferred because of code freeze for an
> upcoming release. You need to either resend after the release, make
> the patch less intrusive, or make a convincing argument that it needs
> to be included in the release.
>
> If you do not understand the reason for this status, disagree with our
> assessment, or are simply not sure how to proceed next, please ask for
> clarification by replying to this email.
>

I don't think this patch has reason to be deferred. It can't possibly 
break anything, as it only syncs the cursor visuals, which is only 
needed because X11 has per-window cursor. Since Windows itself has a 
global cursor, then theoretically the cursor should be "permanently" 
synced for it to have Windows behavior.

Obviously it's prohibitive to do that, but in this case it doesn't 
really break anything other than adding a round-trip to the wineserver 
when a "sizemove" drag operation begins, which needless to say is 
insignificant. If the cursor was already synced then nothing happens.



More information about the wine-devel mailing list