aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authornobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2015-01-08 07:31:45 +0000
committernobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2015-01-08 07:31:45 +0000
commit459894160ec03ba01fc4c4e574480f83604ac410 (patch)
tree3eeebbb132a910a19b0e443287ba0263d1be2a3e
parent759a31b4a3fbacb5a8528a3875c7f2395a2f9187 (diff)
downloadruby-459894160ec03ba01fc4c4e574480f83604ac410.tar.gz
vm_method.c: fix change refined new method visibility
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@49182 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r--ChangeLog4
-rw-r--r--test/ruby/test_refinement.rb14
-rw-r--r--vm_method.c4
3 files changed, 21 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog
index beb167cd0e..f705ab8fe9 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+Thu Jan 8 16:31:43 2015 Seiei Higa <hanachin@gmail.com>
+
+
+
Thu Jan 8 12:53:44 2015 Nobuyoshi Nakada <nobu@ruby-lang.org>
* dir.c (glob_helper): match in case-folding only if the directory
diff --git a/test/ruby/test_refinement.rb b/test/ruby/test_refinement.rb
index 11077b6e4d..8553c42e18 100644
--- a/test/ruby/test_refinement.rb
+++ b/test/ruby/test_refinement.rb
@@ -1165,6 +1165,20 @@ class TestRefinement < Test::Unit::TestCase
end;
end
+ def test_change_refined_new_method_visibility
+ assert_separately([], <<-"end;")
+ bug10706 = '[ruby-core:67387] [Bug #10706]'
+ module RefinementBug
+ refine Object do
+ def foo
+ end
+ end
+ end
+
+ assert_raise(NameError, bug10706) {private(:foo)}
+ end;
+ end
+
private
def eval_using(mod, s)
diff --git a/vm_method.c b/vm_method.c
index 8af2198dbd..97ae64f55d 100644
--- a/vm_method.c
+++ b/vm_method.c
@@ -825,7 +825,9 @@ rb_export_method(VALUE klass, ID name, rb_method_flag_t noex)
me = search_method(rb_cObject, name, &defined_class);
}
- if (UNDEFINED_METHOD_ENTRY_P(me)) {
+ if (UNDEFINED_METHOD_ENTRY_P(me) ||
+ (me->def->type == VM_METHOD_TYPE_REFINED &&
+ UNDEFINED_METHOD_ENTRY_P(me->def->body.orig_me))) {
rb_print_undef(klass, name, 0);
}