diff options
author | shugo <shugo@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-08-23 04:00:25 +0000 |
---|---|---|
committer | shugo <shugo@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-08-23 04:00:25 +0000 |
commit | 55cf24d3789486328e9a2ea734f42fb9122056e9 (patch) | |
tree | b0432f19ef5949373e2885b7d5adfd606154cd73 | |
parent | 742a0d164e127394b438d6b83a8d65b4c16f1981 (diff) | |
download | ruby-55cf24d3789486328e9a2ea734f42fb9122056e9.tar.gz |
* insns.def (invokesuper): reverted r36640 partially to make super
in a thread work correctly. [ruby-core:47284] [Bug #6907]
* test/ruby/test_super.rb: related test.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@36795 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 7 | ||||
-rw-r--r-- | insns.def | 19 | ||||
-rw-r--r-- | test/ruby/test_super.rb | 25 |
3 files changed, 29 insertions, 22 deletions
@@ -1,3 +1,10 @@ +Thu Aug 23 12:51:39 2012 Shugo Maeda <shugo@ruby-lang.org> + + * insns.def (invokesuper): reverted r36640 partially to make super + in a thread work correctly. [ruby-core:47284] [Bug #6907] + + * test/ruby/test_super.rb: related test. + Thu Aug 23 12:30:20 2012 NAKAMURA Usaku <usa@ruby-lang.org> * win32/configure.bat: support --with(out)?-ext(ensions) options. @@ -1026,8 +1026,6 @@ invokesuper int num = caller_setup_args(th, GET_CFP(), flag, (int)op_argc, blockiseq, &blockptr); VALUE recv, klass; - rb_control_frame_t *cfp = GET_CFP(); - rb_control_frame_t *end_cfp = RUBY_VM_END_CONTROL_FRAME(th); ID id; const rb_method_entry_t *me; rb_iseq_t *ip; @@ -1038,20 +1036,9 @@ invokesuper if (!NIL_P(RCLASS_REFINED_CLASS(klass))) { klass = RCLASS_REFINED_CLASS(klass); } - - recv = Qundef; - while (RUBY_VM_VALID_CONTROL_FRAME_P(cfp, end_cfp)) { - if (((VM_EP_LEP_P(cfp->ep) && cfp->iseq && - cfp->iseq->type == ISEQ_TYPE_METHOD) || - (cfp->me && cfp->me->def->type == VM_METHOD_TYPE_BMETHOD)) && - rb_obj_is_kind_of(cfp->self, klass)) { - recv = cfp->self; - break; - } - cfp = RUBY_VM_PREVIOUS_CONTROL_FRAME(cfp); - } - if (recv == Qundef) { - rb_raise(rb_eNoMethodError, "super called outside of method"); + recv = GET_SELF(); + if (!rb_obj_is_kind_of(recv, klass)) { + rb_raise(rb_eNotImpError, "super from singleton method that is defined to multiple classes is not supported; this will be fixed in 2.0.0 or later"); } vm_search_superclass(GET_CFP(), GET_ISEQ(), TOPN(num), &id, &klass); diff --git a/test/ruby/test_super.rb b/test/ruby/test_super.rb index 7208b362fc..dcd700858c 100644 --- a/test/ruby/test_super.rb +++ b/test/ruby/test_super.rb @@ -264,7 +264,9 @@ class TestSuper < Test::Unit::TestCase end } obj = sub_class.new - assert_equal [:super, obj], obj.foo + assert_raise(NotImplementedError) do + obj.foo + end end def test_super_in_instance_eval_with_define_method @@ -282,7 +284,9 @@ class TestSuper < Test::Unit::TestCase end } obj = sub_class.new - assert_equal [:super, obj], obj.foo + assert_raise(NotImplementedError) do + obj.foo + end end def test_super_in_orphan_block @@ -298,9 +302,7 @@ class TestSuper < Test::Unit::TestCase end } obj = sub_class.new - assert_raise(NoMethodError) do - obj.foo.call - end + assert_equal([:super, obj], obj.foo.call) end def test_super_in_orphan_block_with_instance_eval @@ -318,7 +320,7 @@ class TestSuper < Test::Unit::TestCase end } obj = sub_class.new - assert_raise(NoMethodError) do + assert_raise(NotImplementedError) do obj.foo.call end end @@ -334,4 +336,15 @@ class TestSuper < Test::Unit::TestCase } assert_equal 'hi', y.hello end + + def test_super_in_thread + hoge = Class.new { + def bar; 'hoge'; end + } + foo = Class.new(hoge) { + def bar; Thread.new { super }.join.value; end + } + + assert_equal 'hoge', foo.new.bar + end end |