Issue #, if available:
conditional variable :: notify all () must be called only under an acquired lock
Description of changes:
Add lock
Also add unit test for ReaderWriterLock illustrating it is not broken in a heavy multi-threaded write-read.
Check all that applies:
[ ] Did a review by yourself.
[ ] Added proper tests to cover this PR. (If tests are not applicable, explain.)
[ ] Checked if this PR is a breaking (APIs have been changed) change.
[ ] Checked if this PR will not introduce cross-platform inconsistent behavior.
[ ] Checked if this PR would require a ReadMe/Wiki update.
Check which platforms you have built SDK on to verify the correctness of this PR.
[x] Linux
[x] Windows
[ ] Android
[ ] MacOS
[ ] IOS
[ ] Other Platforms
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Issue #, if available: conditional variable :: notify all () must be called only under an acquired lock Description of changes: Add lock Also add unit test for ReaderWriterLock illustrating it is not broken in a heavy multi-threaded write-read. Check all that applies:
Check which platforms you have built SDK on to verify the correctness of this PR.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.