diff options
author | naruse <naruse@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2010-07-21 05:00:53 +0000 |
---|---|---|
committer | naruse <naruse@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2010-07-21 05:00:53 +0000 |
commit | ad717fa7e657703205c44df1b00f7d7d4a60d8ca (patch) | |
tree | 3daaa727391dacfec0ee28be46405645b16fa81c | |
parent | 5647b2cd881bc45e531a71d51c1b9f9bbee9ce7f (diff) | |
download | ruby-ad717fa7e657703205c44df1b00f7d7d4a60d8ca.tar.gz |
* vm_method.c (rb_method_boundp): revert r28543, r28564.
They may be merged in Ruby 2.0. [ruby-core:31217]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@28700 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | test/ruby/test_method.rb | 4 | ||||
-rw-r--r-- | vm_method.c | 21 |
3 files changed, 17 insertions, 13 deletions
@@ -1,3 +1,8 @@ +Wed Jul 21 13:57:37 2010 NARUSE, Yui <naruse@ruby-lang.org> + + * vm_method.c (rb_method_boundp): revert r28543, r28564. + They may be merged in Ruby 2.0. [ruby-core:31217] + Wed Jul 21 13:37:35 2010 Marc-Andre Lafortune <ruby-core@marc-andre.ca> * lib/uri/common.rb: Have URI#route_to, URI#route_from accept diff --git a/test/ruby/test_method.rb b/test/ruby/test_method.rb index d135577208..da17ef5e9c 100644 --- a/test/ruby/test_method.rb +++ b/test/ruby/test_method.rb @@ -371,7 +371,7 @@ class TestMethod < Test::Unit::TestCase assert_equal(true, respond_to?(:mv1)) assert_equal(false, respond_to?(:mv2)) - assert_equal(false, respond_to?(:mv3)) + assert_equal(true, respond_to?(:mv3)) assert_equal(true, respond_to?(:mv1, true)) assert_equal(true, respond_to?(:mv2, true)) @@ -393,7 +393,7 @@ class TestMethod < Test::Unit::TestCase assert_equal(true, v.respond_to?(:mv1)) assert_equal(false, v.respond_to?(:mv2)) - assert_equal(false, v.respond_to?(:mv3)) + assert_equal(true, v.respond_to?(:mv3)) assert_equal(true, v.respond_to?(:mv1, true)) assert_equal(true, v.respond_to?(:mv2, true)) diff --git a/vm_method.c b/vm_method.c index aa5db73643..50f0b12e5a 100644 --- a/vm_method.c +++ b/vm_method.c @@ -565,19 +565,18 @@ rb_method_boundp(VALUE klass, ID id, int ex) { rb_method_entry_t *me = rb_method_entry(klass, id); - if (!me) return 0; - if (ex & ~NOEX_RESPONDS) { /* pub */ - if (me->flag & NOEX_PRIVATE) return 0; - if (ex & NOEX_RESPONDS) { - if (me->flag & NOEX_PROTECTED) return 0; + if (me != 0) { + if ((ex & ~NOEX_RESPONDS) && (me->flag & NOEX_PRIVATE)) { + return FALSE; } + if (!me->def) return 0; + if (me->def->type == VM_METHOD_TYPE_NOTIMPLEMENTED) { + if (ex & NOEX_RESPONDS) return 2; + return 0; + } + return 1; } - if (!me->def) return 0; - if (me->def->type == VM_METHOD_TYPE_NOTIMPLEMENTED) { - if (ex & NOEX_RESPONDS) return 2; - return 0; - } - return 1; + return 0; } void |