Conversation
6773918 to
79a75af
Compare
- inline the take_while_ascii_pred function - change flag handling to stack allocation by using a fixed sized array - Parse regex literals at an earlier opportunity when inspecting expressions
79a75af to
6a7dcdf
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4310 +/- ##
==========================================
+ Coverage 47.24% 50.31% +3.07%
==========================================
Files 476 498 +22
Lines 46892 49924 +3032
==========================================
+ Hits 22154 25120 +2966
- Misses 24738 24804 +66 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I took your suggestion in the issues and made a new function for this issue using an arrayvec which I believe will be marginally more memory efficient in the vast majority of cases (where no flags are given) than allocating an array. Additionally, I added error handling for the case where more than 8 regex flags are added and preserved the original version of the function for future use. However, your pull request covers a lot of area that mine does not we should merge them! |
Uh oh!
There was an error while loading. Please reload this page.