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.
-
Gabriel Ivăncescu