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

Replace most uses of Map by HashMap #60999

Merged
merged 1 commit into from
May 16, 2022
Merged

Replace most uses of Map by HashMap #60999

merged 1 commit into from
May 16, 2022

Commits on May 16, 2022

  1. Replace most uses of Map by HashMap 

    * Map is unnecessary and inefficient in almost every case.
* Replaced by the new HashMap.
* Renamed Map to RBMap and Set to RBSet for cases that still make sense
  (order matters) but use is discouraged.

There were very few cases where replacing by HashMap was undesired because
keeping the key order was intended.
I tried to keep those (as RBMap) as much as possible, but might have missed
some. Review appreciated!
    @reduz @akien-mga
    reduz authored and akien-mga committed May 16, 2022
    Configuration menu
    Copy the full SHA
    746dddc View commit details
    Browse the repository at this point in the history