Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
After running stackprof on my production application homepage I noticed that
TagHelper#tag_option
is the second occurrence:I performed some tests with
benchmark-ips
and noticed thatString#gsub
with a String input is almost 2x faster than with a Regex (in this specific case).I also benchmarked this change and
#tag_option
speed is improved from 20-45% on the most common cases (when escape is true, with or without quotes present) and up to 50% when escape is false.I noticed as well that by freezing the string literals I was able to extract a little bit more of performance when an array of values is passed.
Given that this is an internal method and the diff is very trivial I can't see any reason not to change this and have a little performance improvement for free.