diff options
author | Koichi ITO <koic.ito@gmail.com> | 2020-01-18 00:42:12 +0900 |
---|---|---|
committer | Koichi ITO <koic.ito@gmail.com> | 2020-01-18 00:58:17 +0900 |
commit | 0dde4b4e3055ca918651f974b055e2cf7447fdba (patch) | |
tree | dc7b559682834249173a3b7d62fd2b5c5b120098 /lib/hashie | |
parent | 7fa93b19a098f90019fbe336ffcae4c3dbe4f54f (diff) | |
download | hashie-0dde4b4e3055ca918651f974b055e2cf7447fdba.tar.gz |
Suppress a Ruby's warning when using Ruby 2.6.0+
This PR suppresses the following warning that `deep_merge` method and
`deep_update` method are defined twice when using Ruby 2.6.0+.
```console
% bundle exec rake
(snip)
/Users/koic/src/github.com/hahie/hashie/lib/hashie/mash.rb:226: warning:
method redefined; discarding old deep_merge
/Users/koic/src/github.com/hahie/hashie/lib/hashie/mash.rb:212: warning:
previous definition of deep_merge was here
/Users/koic/src/github.com/hahie/hashie/lib/hashie/mash.rb:232: warning:
method redefined; discarding old deep_update
/Users/koic/src/github.com/hahie/hashie/lib/hashie/mash.rb:218: warning:
previous definition of deep_update was here
```
Diffstat (limited to 'lib/hashie')
-rw-r--r-- | lib/hashie/extensions/ruby_version_check.rb | 6 | ||||
-rw-r--r-- | lib/hashie/mash.rb | 28 |
2 files changed, 19 insertions, 15 deletions
diff --git a/lib/hashie/extensions/ruby_version_check.rb b/lib/hashie/extensions/ruby_version_check.rb index bcd3436..b287230 100644 --- a/lib/hashie/extensions/ruby_version_check.rb +++ b/lib/hashie/extensions/ruby_version_check.rb @@ -9,7 +9,11 @@ module Hashie module ClassMethods def with_minimum_ruby(version) - yield if RubyVersion.new(RUBY_VERSION) >= RubyVersion.new(version) + yield if with_minimum_ruby?(version) + end + + def with_minimum_ruby?(version) + RubyVersion.new(RUBY_VERSION) >= RubyVersion.new(version) end end end diff --git a/lib/hashie/mash.rb b/lib/hashie/mash.rb index 194a6f8..96b24c9 100644 --- a/lib/hashie/mash.rb +++ b/lib/hashie/mash.rb @@ -207,20 +207,7 @@ module Hashie alias include? key? alias member? key? - # Performs a deep_update on a duplicate of the - # current mash. - def deep_merge(other_hash, &blk) - dup.deep_update(other_hash, &blk) - end - - # Recursively merges this mash with the passed - # in hash, merging each hash in the hierarchy. - def deep_update(other_hash, &blk) - _deep_update(other_hash, &blk) - self - end - - with_minimum_ruby('2.6.0') do + if with_minimum_ruby?('2.6.0') # Performs a deep_update on a duplicate of the # current mash. def deep_merge(*other_hashes, &blk) @@ -235,6 +222,19 @@ module Hashie end self end + else + # Performs a deep_update on a duplicate of the + # current mash. + def deep_merge(other_hash, &blk) + dup.deep_update(other_hash, &blk) + end + + # Recursively merges this mash with the passed + # in hash, merging each hash in the hierarchy. + def deep_update(other_hash, &blk) + _deep_update(other_hash, &blk) + self + end end # Alias these lexically so they get the correctly defined |