diff options
-rw-r--r-- | class.c | 21 | ||||
-rw-r--r-- | test/ruby/test_refinement.rb | 32 |
2 files changed, 43 insertions, 10 deletions
@@ -1105,12 +1105,11 @@ include_modules_at(const VALUE klass, VALUE c, VALUE module, int search_super) static enum rb_id_table_iterator_result move_refined_method(ID key, VALUE value, void *data) { - rb_method_entry_t *me = (rb_method_entry_t *) value; - VALUE klass = (VALUE)data; - struct rb_id_table *tbl = RCLASS_M_TBL(klass); + rb_method_entry_t *me = (rb_method_entry_t *)value; if (me->def->type == VM_METHOD_TYPE_REFINED) { - rb_clear_method_cache(klass, me->called_id); + VALUE klass = (VALUE)data; + struct rb_id_table *tbl = RCLASS_M_TBL(klass); if (me->def->body.refined.orig_me) { const rb_method_entry_t *orig_me = me->def->body.refined.orig_me, *new_me; @@ -1130,6 +1129,19 @@ move_refined_method(ID key, VALUE value, void *data) } } +static enum rb_id_table_iterator_result +cache_clear_refined_method(ID key, VALUE value, void *data) +{ + rb_method_entry_t *me = (rb_method_entry_t *) value; + + if (me->def->type == VM_METHOD_TYPE_REFINED) { + VALUE klass = (VALUE)data; + rb_clear_method_cache(klass, me->called_id); + } + + return ID_TABLE_CONTINUE; +} + static void ensure_origin(VALUE klass) { @@ -1141,6 +1153,7 @@ ensure_origin(VALUE klass) RCLASS_SET_ORIGIN(klass, origin); RCLASS_M_TBL(origin) = RCLASS_M_TBL(klass); RCLASS_M_TBL_INIT(klass); + rb_id_table_foreach(RCLASS_M_TBL(origin), cache_clear_refined_method, (void *)klass); rb_id_table_foreach(RCLASS_M_TBL(origin), move_refined_method, (void *)klass); } } diff --git a/test/ruby/test_refinement.rb b/test/ruby/test_refinement.rb index b27a176889..c364de46e4 100644 --- a/test/ruby/test_refinement.rb +++ b/test/ruby/test_refinement.rb @@ -2441,12 +2441,6 @@ class TestRefinement < Test::Unit::TestCase $VERBOSE = verbose_bak end - private - - def eval_using(mod, s) - eval("using #{mod}; #{s}", Sandbox::BINDING) - end - # [Bug #17386] def test_prepended_with_method_cache foo = Class.new do @@ -2473,4 +2467,30 @@ class TestRefinement < Test::Unit::TestCase foo.prepend code assert_equal :Code, obj.foo end + + # [Bug #17417] + def test_prepended_with_method_cache_17417 + assert_normal_exit %q{ + module M + def hoge; end + end + + module R + refine Hash do + def except *args; end + end + end + + h = {} + h.method(:except) # put it on pCMC + Hash.prepend(M) + h.method(:except) + } + end + + private + + def eval_using(mod, s) + eval("using #{mod}; #{s}", Sandbox::BINDING) + end end |