On 01.01.2016 23:50, Christopher Thielen wrote:
Fixes https://bugs.winehq.org/show_bug.cgi?id=13683
A window may be notified with WM_CAPTURECHANGED about itself gaining mouse capture if it calls SetCapture() twice.
Tested on Windows XP and Fedora Workstation 22.
Re-submitting to simplify based on wine-devel feedback. Re-submitting after 1.8 release.
Signed-off-by: Christopher Thielen cthielen@gmail.com
Where are the tests you initially had in your patch? I assume you accidentially resent an old version. The tests are still useful to make sure, that noone will break this fix again.
Also, please review all the occurances of WM_CAPTURECHANGED, and fix them accordingly. If the code doesn't look like it can handle the double notification, then just ignore them based on the condition lParam == hWnd. This could also be done in separate patches before.
Regards, Sebastian
-
Sebastian Lackner