Fixed the `WaitableAtomic` bug with lack of notifications of updates when updating from `Wait()` with predicate + mutating "getter".

C5T / Current

C++ framework for realtime machine learning.

97 stars 29 forks source link

Closed dimacurrentai closed 5 months ago

dimacurrentai commented 5 months ago

I hope the first commit fails the tests, and then I push the fix in the second commit.

If this plan does not work out, I'll increase the "delay" and try the "failing" case again.

Fingers crossed!

dimacurrentai commented 5 months ago

Good, both ubuntu and macos are hung on WaitableAtomic.NotifiesOfChangesInWait, as planned. Pushing the fix.

dimacurrentai commented 5 months ago

@mzhurovich ready for your review!