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

dev: with_capacity for HashMap #7246

Merged
merged 1 commit into from Mar 20, 2024
Merged

dev: with_capacity for HashMap #7246

merged 1 commit into from Mar 20, 2024

Conversation

greged93
Copy link
Contributor

@greged93 greged93 commented Mar 20, 2024 •
edited

Fixes one clippy suggestion and allocates the HashMap in insert_genesis_state using with_capacity.

@mattsse mattsse added this pull request to the merge queue Mar 20, 2024
@mattsse mattsse added the C-perf A change motivated by improving speed, memory usage or disk footprint label Mar 20, 2024
Merged via the queue into paradigmxyz:main with commit c1f051a Mar 20, 2024
27 checks passed
Ruteri pushed a commit to Ruteri/reth that referenced this pull request Apr 17, 2024
@greged93 @Ruteri
dev: with_capacity for HashMap (paradigmxyz#7246)
58a7f34
Ruteri pushed a commit to Ruteri/reth that referenced this pull request Apr 17, 2024
@greged93 @Ruteri
dev: with_capacity for HashMap (paradigmxyz#7246)
733ef23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mattsse mattsse mattsse approved these changes

@rakita rakita Awaiting requested review from rakita rakita is a code owner

@rkrasiuk rkrasiuk Awaiting requested review from rkrasiuk rkrasiuk is a code owner

@gakonst gakonst Awaiting requested review from gakonst

Assignees
No one assigned
Labels
C-perf A change motivated by improving speed, memory usage or disk footprint
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@greged93 @mattsse