⚡ Improve slice allocation efficiency in casing wrappers#11
Conversation
Replaced inefficient double append pattern with single pre-allocated slice creation in ToKebabCase, ToSnakeCase, ToPascalCase, and ToCamelCase. Reduces allocations by 1 per call. Co-authored-by: arran4 <111667+arran4@users.noreply.github.com>
|
👋 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. |
💡 What: Optimized
ToKebabCase,ToSnakeCase,ToPascalCase, andToCamelCaseto use a single slice allocation strategy (make+append) instead of the inefficient doubleappendpattern.🎯 Why: The previous implementation used
append([]Option(nil), append(opts, ...)...), which caused multiple allocations per call. This was redundant and inefficient.📊 Measured Improvement:
Benchmarks show a consistent reduction of 1 allocation per operation across all measured scenarios.
Verified that functionality remains unchanged with existing tests. Added
benchmark_test.goto track performance.PR created automatically by Jules for task 17502699394544383921 started by @arran4