⚡ Optimize StringDehumanizeExtensions with compiled Regex - #84
Conversation
This change replaces inline, uncompiled regexes in ToPascalCase and ToSnakeCase with static readonly fields configured with RegexOptions.Compiled and RegexOptions.CultureInvariant. This avoids the overhead of parsing and compiling the patterns on every call, leading to better performance and fewer allocations. - Added static readonly Regex fields for all patterns. - Updated methods to use the cached instances. - Added RegexOptions.CultureInvariant for consistency with the class's goal. - Added a blank line before the return statement in ToCamelCase to follow style guidelines.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
1 participant
Optimize StringDehumanizeExtensions by caching compiled regexes. Added static readonly Regex fields for all patterns in ToPascalCase and ToSnakeCase, using RegexOptions.Compiled and RegexOptions.CultureInvariant.
PR created automatically by Jules for task 11887808368948327362 started by @winnerspiros