On 2/1/22 15:12, Jacek Caban wrote:
Signed-off-by: Jacek Caban jacek@codeweavers.com
dlls/user32/tests/msg.c | 33 +++++++++++++++++++++++++++++++++ 1 file changed, 33 insertions(+)
Hi Jacek!
I've been trying to fix this for quite some time as I believe we have a race condition in thread destroy and that causes some spurious user32:msg test failures already.
I wrote a few more tests (attached) before to check Windows behavior and they don't pass with your changes.
I also had several variations of the fix, and I'm attaching the last one for reference, although I understand PATCH 3 and 4 which are changing the parent pointer semantics on the wineserver side is a bit risky.
From the errors returned by window functions it doesn't look completely off though (some functions fail in the same way for desktop window and for orphaned windows, both not having a parent).
Note that my fix then doesn't pass your WM_NCDESTROY parent test either.
Cheers,