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

Relax bounds on many methods to be minimal #64

Merged
merged 2 commits into from
Feb 26, 2020
Merged

Relax bounds on many methods to be minimal #64

merged 2 commits into from
Feb 26, 2020

Commits on Feb 26, 2020

  1. Relax bounds on many methods to be minimal 

    This patch removes a large number of unnecessary bounds. Beyond the
obvious ones (like iterators do not require `Hash` and constructors do
not need `BuildHasher`), the biggest change is that the thread-safety
bounds are now _only_ placed on methods that modify the map. The
reasoning here is simple enough: if the threads-safety bounds do not
hold for K/V, then the map must be empty, and if the map is empty, the
read operations are fine even if the K/V type is not thread-safe.
    @jonhoo
    jonhoo committed Feb 26, 2020
    Configuration menu
    Copy the full SHA
    5b48c58 View commit details
    Browse the repository at this point in the history

  2. init_table does not require Clone

    @jonhoo
    jonhoo committed Feb 26, 2020
    Configuration menu
    Copy the full SHA
    cb8f678 View commit details
    Browse the repository at this point in the history