change merge to merge! #9997

Merged
merged 1 commit into from Mar 30, 2013

Conversation

Projects
None yet
2 participants
Member

vipulnsward commented Mar 29, 2013

Change merge to merge! for performance gain[1]

in HashWithIndifferentAccess#.[]

[1] https://gist.github.com/vipulnsward/fa70dec670187713bfd8

@carlosantoniodasilva carlosantoniodasilva added a commit that referenced this pull request Mar 30, 2013

@carlosantoniodasilva carlosantoniodasilva Merge pull request #9997 from vipulnsward/hash_destructive_merge
Change merge to merge! to avoid creating new objects
6a3794b

@carlosantoniodasilva carlosantoniodasilva merged commit 6a3794b into rails:master Mar 30, 2013

vipulnsward deleted the vipulnsward:hash_destructive_merge branch Apr 1, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment