Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Replace ahash with rapidhash for hash tables. #15

Open
Sewer56 opened this issue Dec 4, 2024 · 0 comments
Open

Replace ahash with rapidhash for hash tables. #15

Sewer56 opened this issue Dec 4, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@Sewer56
Copy link
Owner

Sewer56 commented Dec 4, 2024

Relevant line of code (table_of_contents_builder_state.rs)

pub(crate) relative_path_to_index: HashMap<&'a str, u32, RandomState>,

Relevant:

In both smhasher and my own oneshot testing, rapidhash performs better for short string keys; and is also portable.
Seems to be a no brainer here.
@Sewer56 Sewer56 added the enhancement New feature or request label Dec 4, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Sewer56