Project

General

Profile

Actions
Copy link

Patch #34160

closed

Replace Hash#merge! with Hash#[]=

Added by Go MAEDA over 3 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Go MAEDA
Category:
Performance
Target version:
4.2.0
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Hash#[]= is faster than Hash#merge!.

https://docs.rubocop.org/rubocop-performance/cops_performance.html#performanceredundantmerge
https://github.com/JuanitoFatas/fast-ruby#hashmerge-vs-hash-code

Replacement can be done with RuboCop:

bundle exec rubocop --only Performance/RedundantMerge -a
Actions
Copy link
 #1

Updated by Go MAEDA over 3 years ago

  • Status changed from New to Closed

Done.

Actions
Copy link
 #2

Updated by Go MAEDA over 3 years ago

  • Assignee set to Go MAEDA
Actions
Copy link

Also available in: Atom PDF