Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Properly handle the race condition #164

Merged
merged 2 commits into from
Nov 25, 2022
Merged

Properly handle the race condition #164

merged 2 commits into from
Nov 25, 2022

Commits on Nov 20, 2022

  1. Properly handle the race condition 

    Use synchronization primitives instead of sleeping.
This eliminates the need for the timeout parameter.
The affected class is effectively internal, so this "should" not affect
regular users.

Also fixed the tests: they didn't clean up after themselves.

Added benchmarks.

Fixes #152
    @mpenkov
    mpenkov committed Nov 20, 2022
    Configuration menu
    Copy the full SHA
    a1ef236 View commit details
    Browse the repository at this point in the history

Commits on Nov 21, 2022

  1. fix comment

    @mpenkov
    mpenkov committed Nov 21, 2022
    Configuration menu
    Copy the full SHA
    5d3bb03 View commit details
    Browse the repository at this point in the history