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.

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

perf: Replace HashMap with a Vec #241

Merged
merged 1 commit into from Nov 10, 2023
Merged

Commits on Nov 10, 2023

  1. perf: Replace HashMap with a Vec

    This may be a minor startup perf win (as `env_logger` does not need the
    expense of DoS resistant hashes) but my main reason for doing this is
    that the default hashing algorithm depends on random number generation,
    which may fail.
    This is important to avoid especially because `env_logger` is often used
    on program startup.
    This was a real issue on Windows.
    While that specific issue is now fixed,
    it would be nice to avoid this class of issues if possible.
    ChrisDenton authored and epage committed Nov 10, 2023
    Configuration menu
    Copy the full SHA
    46ccdd9 View commit details
    Browse the repository at this point in the history