diff --git a/ChangeLog b/ChangeLog index 991148f3cb..a0ed168061 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,18 +1,3 @@ -Mon Jul 29 16:48:38 2013 Nobuyoshi Nakada - - * hash.c (rb_hash_initialize_copy): copy st_table type even if empty. - [ruby-core:56256] [Bug #8703] - -Mon Jul 29 16:34:29 2013 Nobuyoshi Nakada - - * hash.c (rb_hash_initialize_copy): clear old table before copy new - table. - -Mon Jul 29 16:34:09 2013 Nobuyoshi Nakada - - * hash.c (rb_hash_assoc): aggregate object can be initialized only - with link time constants. - Mon Jul 29 14:54:44 2013 Masaki Matsushita * hash.c (rb_hash_assoc): performance improvement by replacing diff --git a/hash.c b/hash.c index d346a520aa..5a8dd9b1fa 100644 --- a/hash.c +++ b/hash.c @@ -1306,14 +1306,22 @@ replace_i(VALUE key, VALUE val, VALUE hash) static VALUE rb_hash_initialize_copy(VALUE hash, VALUE hash2) { + st_table *ntbl; + rb_hash_modify_check(hash); hash2 = to_hash(hash2); Check_Type(hash2, T_HASH); - if (!RHASH_EMPTY_P(hash2)) { + ntbl = RHASH(hash)->ntbl; + if (RHASH(hash2)->ntbl) { + if (ntbl) st_free_table(ntbl); RHASH(hash)->ntbl = st_copy(RHASH(hash2)->ntbl); - rb_hash_rehash(hash); + if (RHASH(hash)->ntbl->num_entries) + rb_hash_rehash(hash); + } + else if (ntbl) { + st_clear(ntbl); } if (FL_TEST(hash2, HASH_PROC_DEFAULT)) { @@ -2167,11 +2175,10 @@ rb_hash_assoc(VALUE hash, VALUE key) struct lookup2_arg arg; struct reset_hash_type_arg ensure_arg; const struct st_hash_type *orighash = table->type; - const struct st_hash_type assochash = { - assoc_cmp, - orighash->hash, - }; + struct st_hash_type assochash; + assochash.compare = assoc_cmp; + assochash.hash = orighash->hash; table->type = &assochash; arg.hash = hash; arg.key = key; diff --git a/test/ruby/test_hash.rb b/test/ruby/test_hash.rb index 1baff2982c..e8b8ef040c 100644 --- a/test/ruby/test_hash.rb +++ b/test/ruby/test_hash.rb @@ -102,6 +102,12 @@ class TestHash < Test::Unit::TestCase assert_raises(TypeError) { h.dup } end + def test_clear_initialize_copy + h = @cls[1=>2] + h.instance_eval {initialize_copy({})} + assert_empty(h) + end + def test_dup_will_rehash set1 = { } set2 = { set1 => true} @@ -912,13 +918,17 @@ class TestHash < Test::Unit::TestCase def test_compare_by_identity a = "foo" - assert_not_predicate({}, :compare_by_identity?) - h = { a => "bar" } + assert_not_predicate(@cls[], :compare_by_identity?) + h = @cls[a => "bar"] assert_not_predicate(h, :compare_by_identity?) h.compare_by_identity assert_predicate(h, :compare_by_identity?) #assert_equal("bar", h[a]) assert_nil(h["foo"]) + + bug8703 = '[ruby-core:56256] [Bug #8703] copied identhash' + h.clear + assert_predicate(h.dup, :compare_by_identity?, bug8703) end class ObjWithHash