Patch #34161: Replace gsub with tr, delete, or squeeze - Redmine

Actions

Patch #34161

closed

Replace gsub with tr, delete, or squeeze

Added by Go MAEDA almost 5 years ago. Updated almost 5 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Category:

Performance

Target version:

Start date:

Due date:

% Done:

0%

Estimated time:

Description

String#tr, String#delete, and String#squeeze are faster than String#gsub.

https://docs.rubocop.org/rubocop-performance/cops_performance.html#performancestringreplacement
https://github.com/JuanitoFatas/fast-ruby#stringgsub-vs-stringtr-code
https://docs.rubocop.org/rubocop-performance/cops_performance.html#performancesqueeze
https://github.com/JuanitoFatas/fast-ruby#remove-extra-spaces-or-other-contiguous-characters-code

Replacement can be done with RuboCop:

bundle exec rubocop --only Performance/StringReplacement,Performance/Squeeze -a

Actions

#1

Updated by Go MAEDA almost 5 years ago

Subject changed from Replace gsub with tr or delete to Replace gsub with tr, delete, or squeeze
Description updated (diff)

Actions

#2

Updated by Go MAEDA almost 5 years ago

Status changed from New to Closed
Assignee set to Go MAEDA

Committed.

Actions

Also available in: Atom PDF