chore: add normalization benchmarks with additional workload scenarios#31
chore: add normalization benchmarks with additional workload scenarios#31
Conversation
Merging this PR will not alter performance
Performance Changes
Comparing Footnotes
|
This comment was marked as outdated.
This comment was marked as outdated.
There was a problem hiding this comment.
Pull request overview
This PR expands the existing Criterion benchmarks for SugarPath::normalize() to cover additional workload scenarios (already-normalized inputs, mixed workloads, and size/depth-focused cases) so performance characteristics can be compared across different path shapes.
Changes:
- Split the prior single
normalizebenchmark into multiple targeted benchmark cases (needs-work, already-clean, mixed workload, short clean, deep clean). - Extended bench fixture sets with more normalization-heavy strings and additional absolute-path samples.
- Added OS-specific “already normalized” fixture lists for Unix and Windows.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| benches/normalize.rs | Adds several new normalize_* benchmarks targeting distinct workload patterns. |
| benches/fixtures.rs | Adds additional path fixtures, including OS-specific “already normalized” path lists used by the new benchmarks. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.