chopraanmol1 / gmail.com wrote:
> I think so far it looks good, let me know if I did something wrong while running the above benchmark.

I agree, so I've committed your patch as-is for r65371.

I wanted to try a shorter patch:

https://80x24.org/spew/20181026050908.1183-1-e / 80x24.org/raw

But I got some spec failures due to singleton class (below).  I
haven't investigated, yet, but I think there may be an existing
bug in hash.c, because my shorter patch ought to work...

1)
Hash#[]= duplicates string keys using dup semantics FAILED
Expected "bar"
 to equal "oof"

ruby/spec/ruby/core/hash/shared/store.rb:16:in `block (2 levels) in <top (required)>'
ruby/spec/ruby/core/hash/element_set_spec.rb:5:in `<top (required)>'

2)
Hash#store duplicates string keys using dup semantics FAILED
Expected "bar"
 to equal "oof"

ruby/spec/ruby/core/hash/shared/store.rb:16:in `block (2 levels) in <top (required)>'
ruby/spec/ruby/core/hash/store_spec.rb:5:in `<top (required)>'

Unsubscribe: <mailto:ruby-core-request / ruby-lang.org?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>