ruby-changes:11085
From: knu <ko1@a...>
Date: Sun, 1 Mar 2009 03:18:04 +0900 (JST)
Subject: [ruby-changes:11085] Ruby:r22679 (ruby_1_8): * class.c (rb_singleton_class_clone): Qnil must be used for a null
knu 2009-03-01 03:12:15 +0900 (Sun, 01 Mar 2009) New Revision: 22679 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=22679 Log: * class.c (rb_singleton_class_clone): Qnil must be used for a null class reference when we use NIL_P() to check class reference validity. The bug was exposed by the spec test of Sequel. * eval.c (ruby_init): Use NEW_CREF(). Modified files: branches/ruby_1_8/ChangeLog branches/ruby_1_8/class.c branches/ruby_1_8/eval.c Index: ruby_1_8/class.c =================================================================== --- ruby_1_8/class.c (revision 22678) +++ ruby_1_8/class.c (revision 22679) @@ -153,7 +153,7 @@ data.klass = obj; break; default: - data.klass = 0; + data.klass = Qnil; break; } Index: ruby_1_8/ChangeLog =================================================================== --- ruby_1_8/ChangeLog (revision 22678) +++ ruby_1_8/ChangeLog (revision 22679) @@ -1,3 +1,11 @@ +Sun Mar 1 03:04:19 2009 Akinori MUSHA <knu@i...> + + * class.c (rb_singleton_class_clone): Qnil must be used for a null + class reference when we use NIL_P() to check class reference + validity. The bug was exposed by the spec test of Sequel. + + * eval.c (ruby_init): Use NEW_CREF(). + Sat Feb 28 18:05:58 2009 Nobuyoshi Nakada <nobu@r...> * eval.c (rb_eval): needs to guard intermediate string objects. Index: ruby_1_8/eval.c =================================================================== --- ruby_1_8/eval.c (revision 22678) +++ ruby_1_8/eval.c (revision 22679) @@ -1434,7 +1434,7 @@ rb_call_inits(); ruby_class = rb_cObject; ruby_frame->self = ruby_top_self; - ruby_top_cref = rb_node_newnode(NODE_CREF,rb_cObject,0,0); + ruby_top_cref = NEW_CREF(rb_cObject, 0); ruby_cref = ruby_top_cref; rb_define_global_const("TOPLEVEL_BINDING", rb_f_binding(ruby_top_self)); #ifdef __MACOS__ -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/