Skip to content

perf: use Vec::with_capacity for hooks and plugins#10178

Merged
chenjiahan merged 1 commit intoweb-infra-dev:mainfrom
CPunisher:04-25-perf/hook-vec
Apr 25, 2025
Merged

perf: use Vec::with_capacity for hooks and plugins#10178
chenjiahan merged 1 commit intoweb-infra-dev:mainfrom
CPunisher:04-25-perf/hook-vec

Conversation

@CPunisher
Copy link
Contributor

Summary

Just an occasional discovery when doing other works.

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@github-actions github-actions bot added the release: performance release: performance related release(mr only) label Apr 25, 2025
@netlify
Copy link

netlify bot commented Apr 25, 2025

Deploy Preview for rspack canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit 57dd257
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/680b2665fd94410008ae8772

@chenjiahan chenjiahan requested a review from ahabhgk April 25, 2025 06:09
@codspeed-hq
Copy link

codspeed-hq bot commented Apr 25, 2025

CodSpeed Performance Report

Merging #10178 will not alter performance

Comparing CPunisher:04-25-perf/hook-vec (57dd257) with main (ad6d0de)

🎉 Hooray! codspeed-rust just leveled up to 2.7.2!

A heads-up, this is a breaking change and it might affect your current performance baseline a bit. But here's the exciting part - it's packed with new, cool features and promises improved result stability 🥳!
Curious about what's new? Visit our releases page to delve into all the awesome details about this new version.

Summary

✅ 11 untouched benchmarks

@chenjiahan chenjiahan merged commit abcebfe into web-infra-dev:main Apr 25, 2025
38 checks passed
@CPunisher CPunisher deleted the 04-25-perf/hook-vec branch April 25, 2025 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: performance release: performance related release(mr only)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants