Skip to content

Commit 22696a0

Browse files
committed
Make WaitForMultipleEvents atomic when waitAll is true
This patch significantly changes the behavior (and to an extent, the performance) of pevents to more closely mimic the documented behavior of `WaitForMultipleObjects` and `WaitForMultipleObjectsEx`. As reported in #9, the previous behavior did not make any atomicity guarantees when a call to `WaitForMultipleEvents()` was made with `waitAll = true`, and WFME would attempt to serially obtain the events in question, which could lead to a deadlock in case of circular locking and auto-reset events. The WFMO behavior documented on MSDN makes it clear that the Windows implementation does not modify the signalled state of any of the manual or auto reset events being awaited until the WFMO call is ready to return, at which point either the one event in question or all the events being awaited (dependent on `waitAll`) are atomically awaited.
1 parent 18dc0e5 commit 22696a0

File tree

1 file changed

+179
-133
lines changed

1 file changed

+179
-133
lines changed

0 commit comments

Comments
 (0)