From 5d6440c74847135962d87323bbdc71fe342c4c51 Mon Sep 17 00:00:00 2001 From: matz Date: Fri, 23 Oct 2009 23:37:14 +0000 Subject: * class.c (rb_class_new): move class check to rb_check_inheritable(). * class.c (rb_check_inheritable): should not allow subclass of class Class. [ruby-core:26225] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@25449 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 7 +++++++ class.c | 6 +++--- test/ruby/test_class.rb | 3 +-- 3 files changed, 11 insertions(+), 5 deletions(-) diff --git a/ChangeLog b/ChangeLog index cdb8245c80..6e829ade40 100644 --- a/ChangeLog +++ b/ChangeLog @@ -3,6 +3,13 @@ Sat Oct 24 00:36:47 2009 Tanaka Akira * io.c (io_cntl): update max file descriptor by the result of fcntl(F_DUPFD). +Fri Oct 23 16:31:14 2009 Yukihiro Matsumoto + + * class.c (rb_class_new): move class check to rb_check_inheritable(). + + * class.c (rb_check_inheritable): should not allow subclass of + class Class. [ruby-core:26225] + Fri Oct 23 14:25:54 2009 Nobuyoshi Nakada * configure.in (target, target_alias): replace with real cpu. diff --git a/class.c b/class.c index f8407ca77b..351f7aeb32 100644 --- a/class.c +++ b/class.c @@ -97,6 +97,9 @@ rb_check_inheritable(VALUE super) if (RBASIC(super)->flags & FL_SINGLETON) { rb_raise(rb_eTypeError, "can't make subclass of singleton class"); } + if (super == rb_cClass) { + rb_raise(rb_eTypeError, "can't make subclass of Class"); + } } @@ -111,9 +114,6 @@ rb_class_new(VALUE super) { Check_Type(super, T_CLASS); rb_check_inheritable(super); - if (super == rb_cClass) { - rb_raise(rb_eTypeError, "can't make subclass of Class"); - } return rb_class_boot(super); } diff --git a/test/ruby/test_class.rb b/test/ruby/test_class.rb index 8fe1272314..a1f087ad63 100644 --- a/test/ruby/test_class.rb +++ b/test/ruby/test_class.rb @@ -170,8 +170,7 @@ class TestClass < Test::Unit::TestCase o = Object.new c = class << o; self; end assert_raise(TypeError) { Class.new(c) } - - assert_nothing_raised { Class.new(Class) } # is it OK? + assert_raise(TypeError) { Class.new(Class) } assert_raise(TypeError) { eval("class Foo < Class; end") } end -- cgit v1.2.3