From 4707fdea0171aab43a1178d4344725d8ca145ada Mon Sep 17 00:00:00 2001 From: shugo Date: Thu, 10 Jan 2013 07:51:35 +0000 Subject: * vm_insnhelper.c (vm_search_super_method): raise a TypeError instead of a NotImplementError if self is not an instance of the current class. [ruby-dev:39772] [Bug #2402] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@38761 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 6 ++++++ test/ruby/test_super.rb | 6 +++--- vm_insnhelper.c | 8 +++++++- 3 files changed, 16 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index f744c4c1b4..f225fcc6ae 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +Thu Jan 10 16:31:20 2013 Shugo Maeda + + * vm_insnhelper.c (vm_search_super_method): raise a TypeError + instead of a NotImplementError if self is not an instance of the + current class. [ruby-dev:39772] [Bug #2402] + Thu Jan 10 16:47:18 2013 Nobuyoshi Nakada * ext/tk/extconf.rb (find_tcltk_header): use have_header instead of diff --git a/test/ruby/test_super.rb b/test/ruby/test_super.rb index bb1d46c636..9505dccdd1 100644 --- a/test/ruby/test_super.rb +++ b/test/ruby/test_super.rb @@ -265,7 +265,7 @@ class TestSuper < Test::Unit::TestCase end } obj = sub_class.new - assert_raise(NotImplementedError) do + assert_raise(TypeError) do obj.foo end end @@ -285,7 +285,7 @@ class TestSuper < Test::Unit::TestCase end } obj = sub_class.new - assert_raise(NotImplementedError) do + assert_raise(TypeError) do obj.foo end end @@ -321,7 +321,7 @@ class TestSuper < Test::Unit::TestCase end } obj = sub_class.new - assert_raise(NotImplementedError) do + assert_raise(TypeError) do obj.foo.call end end diff --git a/vm_insnhelper.c b/vm_insnhelper.c index 6e00edde07..d82ffb3d1d 100644 --- a/vm_insnhelper.c +++ b/vm_insnhelper.c @@ -1943,7 +1943,13 @@ vm_search_super_method(rb_thread_t *th, rb_control_frame_t *reg_cfp, rb_call_inf if (!FL_TEST(current_defined_class, RMODULE_INCLUDED_INTO_REFINEMENT) && !rb_obj_is_kind_of(ci->recv, current_defined_class)) { - 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"); + VALUE m = RB_TYPE_P(current_defined_class, T_ICLASS) ? + RBASIC(current_defined_class)->klass : current_defined_class; + + rb_raise(rb_eTypeError, + "self has wrong type to call super in this context: " + "%s (expected %s)", + rb_obj_classname(ci->recv), rb_class2name(m)); } vm_search_superclass(GET_CFP(), iseq, sigval, ci); -- cgit v1.2.3