fixup! Make WaitForMultipleEvents atomic when waitAll is true

neosmart · Jul 8, 2021 · 1daad02 · 1daad02
1 parent 2db23ff
commit 1daad02
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/pevents.cpp b/src/pevents.cpp
@@ -446,6 +446,12 @@ namespace neosmart {
                         consumed = true;
                     }
 
+                    // Explictly set `StillWaiting` to false ourselves rather than waiting for the
+                    // waiter to do it so another `SetEvent()` on a different autoreset event also
+                    // being awaited by the same waiter isn't also consumed if scheduled before the
+                    // waiter gets around to setting StillWaiting to false.
+                    i->Waiter->StillWaiting = false;
+
                     result = pthread_cond_signal(&i->Waiter->CVariable);
                     assert(result == 0);
                 }