perf: use custom canonicalize impl to avoid usless syscall

[Brooooooklyn]

Reduce one redundant fs::symlink_metadata system call.

[Brooooooklyn]

• perf: use custom canonicalize impl to avoid usless syscall #220 👈
• bench: add symlink fixtures #219
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Brooooooklyn and the rest of your teammates on Graphite

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.95%. Comparing base (1013c5e) to head (6f70506).

Additional details and impacted files

@@                         Coverage Diff                          @@
##           07-09-bench_add_symlink_fixtures     #220      +/-   ##
====================================================================
+ Coverage                             96.92%   96.95%   +0.03%     
====================================================================
  Files                                    11       11              
  Lines                                  2306     2298       -8     
====================================================================
- Hits                                   2235     2228       -7     
+ Misses                                   71       70       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[codspeed-hq]

CodSpeed Performance Report

Merging #220 will not alter performance

_{Comparing 07-09-perf_use_custom_canonicalize_impl_to_avoid_heavy_syscall (a5fc16f) with main (e29975d)}

Summary

✅ 3 untouched benchmarks

[Boshen]

Request me a review once you're done.

[Boshen]

Merge activity

• Jul 10, 10:33 AM EDT: @Boshen started a stack merge that includes this pull request via Graphite.
• Jul 10, 10:35 AM EDT: Graphite rebased this pull request as part of a merge.
• Jul 10, 10:37 AM EDT: @Boshen merged this pull request with Graphite.