From fa3affad9a7677cf0307fbb218b612e2626ee9f3 Mon Sep 17 00:00:00 2001 From: ko1 Date: Wed, 18 Nov 2015 13:59:57 +0000 Subject: [PATCH] * vm_method.c (rb_class_clear_method_cache): should clear all RCLASS_CALLABLE_M_TBLs of all sub-classes (T_ICLASS). RCLASS_CALLABLE_M_TBL() caches complemented method entries. It should be cleared when the modules are cleared. On previous version clears only for direct children. It is enough for normal modules because corresponding T_ICLASSes are direct children. However, refinements create complex data structure. So that we need to clear all children (and descendants). [ruby-core:71423] [Bug #11672] * vm_method.c (rb_clear_method_cache_by_class): rb_mKernel doesn't call rb_class_clear_method_cache, so that clear child T_ICLASSes. * test/ruby/test_refinement.rb: enable disabled test. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@52648 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 21 +++++++++++++++++++++ test/ruby/test_refinement.rb | 6 ------ vm_method.c | 26 ++++++++++++++++++++------ 3 files changed, 41 insertions(+), 12 deletions(-) diff --git a/ChangeLog b/ChangeLog index b0cbc939e8..7ce0bda252 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,24 @@ +Wed Nov 18 22:50:43 2015 Koichi Sasada + + * vm_method.c (rb_class_clear_method_cache): should clear all + RCLASS_CALLABLE_M_TBLs of all sub-classes (T_ICLASS). + + RCLASS_CALLABLE_M_TBL() caches complemented method entries. + It should be cleared when the modules are cleared. + On previous version clears only for direct children. + It is enough for normal modules because corresponding T_ICLASSes + are direct children. + + However, refinements create complex data structure. So that + we need to clear all children (and descendants). + [ruby-core:71423] [Bug #11672] + + * vm_method.c (rb_clear_method_cache_by_class): rb_mKernel + doesn't call rb_class_clear_method_cache, so that + clear child T_ICLASSes. + + * test/ruby/test_refinement.rb: enable disabled test. + Wed Nov 18 21:09:08 2015 Koichi Sasada * vm_method.c (prepare_callable_method_entry): use diff --git a/test/ruby/test_refinement.rb b/test/ruby/test_refinement.rb index b69135f321..43f58b1038 100644 --- a/test/ruby/test_refinement.rb +++ b/test/ruby/test_refinement.rb @@ -1495,7 +1495,6 @@ class TestRefinement < Test::Unit::TestCase end def test_reopen_refinement_module - flag = false assert_separately([], <<-"end;") $VERBOSE = nil class C @@ -1522,11 +1521,6 @@ class TestRefinement < Test::Unit::TestCase assert_equal(:bar, C.new.m, "[ruby-core:71423] [Bug #11672]") end; - flag = true - rescue MiniTest::Assertion - skip 'expected to fail' - ensure - raise MiniTest::Assertion, 'this test is expected to fail' if flag end private diff --git a/vm_method.c b/vm_method.c index 5089c31ee4..eff6b944c2 100644 --- a/vm_method.c +++ b/vm_method.c @@ -62,6 +62,20 @@ static void rb_class_clear_method_cache(VALUE klass, VALUE arg) { RCLASS_SERIAL(klass) = rb_next_class_serial(); + + if (RB_TYPE_P(klass, T_ICLASS)) { + struct rb_id_table *table = RCLASS_CALLABLE_M_TBL(klass); + if (table) { + rb_id_table_clear(table); + } + } + else { + if (RCLASS_CALLABLE_M_TBL(klass) != 0) { + rb_obj_info_dump(klass); + rb_bug("RCLASS_CALLABLE_M_TBL(klass) != 0"); + } + } + rb_class_foreach_subclass(klass, rb_class_clear_method_cache, arg); } @@ -93,14 +107,14 @@ rb_clear_method_cache_by_class(VALUE klass) else { rb_class_clear_method_cache(klass, Qnil); } + } - if (RB_TYPE_P(klass, T_MODULE)) { - rb_subclass_entry_t *entry = RCLASS_EXT(klass)->subclasses; + if (klass == rb_mKernel) { + rb_subclass_entry_t *entry = RCLASS_EXT(klass)->subclasses; - for (; entry != NULL; entry = entry->next) { - struct rb_id_table *table = RCLASS_CALLABLE_M_TBL(entry->klass); - if (table)rb_id_table_clear(table); - } + for (; entry != NULL; entry = entry->next) { + struct rb_id_table *table = RCLASS_CALLABLE_M_TBL(entry->klass); + if (table)rb_id_table_clear(table); } } }