April 29, 2026
1:52 p.m.
On Wed Apr 29 02:52:20 2026 +0000, Larry Starr wrote:
My experience with windows programming is limited, but my understanding is we need them to ensure the gui has time to finish the drawing before we then go and check things. Test appears to work as is so I do not see a need to remove them. Do you see any issue with them? flush_events() is mostly needed when we need to wait for things like x11 events or rendering things on screen like you said. However, I don't think this is either of those cases. You are testing win32 states, which should be the same without `flush_events()`. I tried removing them and the tests run fine. I just don't see the need for them.
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/10340#note_138195