[前][次][番号順一覧][スレッド一覧]

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/

[前][次][番号順一覧][スレッド一覧]