Repository navigation
performance improvements - #355
anastygnome wants to merge 8 commits into
Conversation
Choose the date parser from the first character instead of trying every form.
d8861cd to
afd20cd
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #355 +/- ##
==========================================
+ Coverage 98.90% 98.98% +0.08%
==========================================
Files 21 21
Lines 4546 4714 +168
Branches 142 144 +2
==========================================
+ Hits 4496 4666 +170
+ Misses 49 47 -2
Partials 1 1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Merging this PR will improve performance by 16.98%1 warning
Performance Changes
Tip Curious why performance improved? Comment Comparing |
|
I really like you didn't touch tests :) |
|
I did ? |
well, I just rewrote the parser in place and rechecked every rule, want me to add some? |
322c531 to
7857ca0
Compare
Parse shared time and offset prefixes once to reduce backtracking in common ISO inputs.
Choose the keyword or numeric path from the first character instead of trying failed alternatives.
Route text and numeric ordinals separately before parsing either form.
Skip the separated-date parser when the input is clearly a compact ISO date.
Avoid probing the TZ rule parser when the input does not start with the TZ prefix.
Use the crate-level limit in year parsing instead of keeping a second copy.
7857ca0 to
178cb8a
Compare
Depends on #350