Re: [PATCH] user32: Allow WM_CAPTURECHANGE indicating SetCapture() parameter. (try 2)

[Sebastian Lackner]

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(a)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