diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-09-09 07:56:02 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-09-09 07:56:02 +0000 |
commit | a71ee2ce41e7f698cdc01d59816de6fe24775850 (patch) | |
tree | 7cc251829d53cd760836696526ce325f20787276 | |
parent | d968e9ecfa3b5b8e7ef801031571cae69de39209 (diff) | |
download | ruby-a71ee2ce41e7f698cdc01d59816de6fe24775850.tar.gz |
vm_method.c: clear cache after refine method
* vm_method.c (rb_add_refined_method_entry): clear cache in the
refined class since refining a method entry is modifying the class.
[ruby-core:57079] [Bug #8880]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@42896 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | test/ruby/test_refinement.rb | 23 | ||||
-rw-r--r-- | vm_method.c | 1 |
3 files changed, 30 insertions, 0 deletions
@@ -1,3 +1,9 @@ +Mon Sep 9 16:55:59 2013 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * vm_method.c (rb_add_refined_method_entry): clear cache in the + refined class since refining a method entry is modifying the class. + [ruby-core:57079] [Bug #8880] + Mon Sep 9 09:14:58 2013 Nobuyoshi Nakada <nobu@ruby-lang.org> * tool/rbinstall.rb (Gem::Specification#initialize): default date to diff --git a/test/ruby/test_refinement.rb b/test/ruby/test_refinement.rb index 8efbb1a85a..02193ae615 100644 --- a/test/ruby/test_refinement.rb +++ b/test/ruby/test_refinement.rb @@ -982,6 +982,29 @@ class TestRefinement < Test::Unit::TestCase RUBY end + def test_refine_after_using + assert_separately([], <<-"end;") + bug8880 = '[ruby-core:57079] [Bug #8880]' + module Test + refine(String) do + end + end + using Test + def t + 'Refinements are broken!'.chop! + end + t + module Test + refine(String) do + def chop! + self.sub!(/broken/, 'fine') + end + end + end + assert_equal('Refinements are fine!', t, bug8880) + end; + end + private def eval_using(mod, s) diff --git a/vm_method.c b/vm_method.c index 41064ad9b4..35c112aea1 100644 --- a/vm_method.c +++ b/vm_method.c @@ -190,6 +190,7 @@ rb_add_refined_method_entry(VALUE refined_class, ID mid) if (me) { make_method_entry_refined(me); + rb_clear_cache_by_class(refined_class); } else { rb_add_method(refined_class, mid, VM_METHOD_TYPE_REFINED, 0, |