diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2009-09-25 04:04:48 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2009-09-25 04:04:48 +0000 |
commit | 66a97945e30119503b43d0d2b8dde62c961c3c17 (patch) | |
tree | d18b20c50bb071a6c71b21ac457f54935edd5435 | |
parent | 8be423f51d56f92cf8a16fab02bcfebda5c409c0 (diff) |
* proc.c (mnew): fix for instance method of Module, BasicObject
and subclass of a class which overrides respond_to_missing?.
based on a patch from Nikolai Lugovoi <nlugovoi AT gmail.com> in
[ruby-core:25748].
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@25089 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 7 | ||||
-rw-r--r-- | proc.c | 2 | ||||
-rw-r--r-- | test/ruby/test_object.rb | 15 |
3 files changed, 21 insertions, 3 deletions
@@ -1,3 +1,10 @@ +Fri Sep 25 13:04:46 2009 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * proc.c (mnew): fix for instance method of Module, BasicObjec + and subclass of a class which overrides respond_to_missing?. + based on a patch from Nikolai Lugovoi <nlugovoi AT gmail.com> in + [ruby-core:25748]. + Fri Sep 25 11:56:50 2009 Nobuyoshi Nakada <nobu@ruby-lang.org> * vm_method.c (rb_mod_method_defined): should return true or false. @@ -912,7 +912,7 @@ mnew(VALUE klass, VALUE obj, ID id, VALUE mclass, int scope) ID rmiss = rb_intern("respond_to_missing?"); VALUE sym = ID2SYM(id); - if (!rb_method_basic_definition_p(klass, rmiss)) { + if (obj != Qundef && !rb_method_basic_definition_p(klass, rmiss)) { if (RTEST(rb_funcall(obj, rmiss, 1, sym))) { return rb_proc_new(missing_wrap, rb_assoc_new(obj, sym)); } diff --git a/test/ruby/test_object.rb b/test/ruby/test_object.rb index 47e4a640d9..37317cfd47 100644 --- a/test/ruby/test_object.rb +++ b/test/ruby/test_object.rb @@ -305,8 +305,7 @@ class TestObject < Test::Unit::TestCase end def test_respond_to_missing - c = Class.new - c.class_eval do + c = Class.new do def respond_to_missing?(id) if id == :foobar true @@ -335,6 +334,18 @@ class TestObject < Test::Unit::TestCase foobar = foo.method(:foobar) assert_equal([:foo], foobar.call); assert_equal([:foo, 1], foobar.call(1)); + + c = Class.new(c) + assert_equal(false, c.method_defined?(:foobar)) + assert_raise(NameError, '[ruby-core:25748]') do + c.instance_method(:foobar) + end + + m = Module.new + assert_equal(false, m.method_defined?(:foobar)) + assert_raise(NameError, '[ruby-core:25748]') do + m.instance_method(:foobar) + end end def test_send_with_no_arguments |