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

ruby-changes:5316

From: shyouhei <ko1@a...>
Date: Wed, 4 Jun 2008 19:17:45 +0900 (JST)
Subject: [ruby-changes:5316] Ruby:r16816 (ruby_1_8_6): merge revision(s) 13586:

shyouhei	2008-06-04 19:17:30 +0900 (Wed, 04 Jun 2008)

  New Revision: 16816

  Modified files:
    branches/ruby_1_8_6/ChangeLog
    branches/ruby_1_8_6/eval.c
    branches/ruby_1_8_6/version.h

  Log:
    merge revision(s) 13586:
    * eval.c (remove_method): should not remove undef place holder.
      [ruby-dev:31817]


  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/branches/ruby_1_8_6/ChangeLog?r1=16816&r2=16815&diff_format=u
  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/branches/ruby_1_8_6/version.h?r1=16816&r2=16815&diff_format=u
  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/branches/ruby_1_8_6/eval.c?r1=16816&r2=16815&diff_format=u

Index: ruby_1_8_6/ChangeLog
===================================================================
--- ruby_1_8_6/ChangeLog	(revision 16815)
+++ ruby_1_8_6/ChangeLog	(revision 16816)
@@ -14,6 +14,11 @@
 	* gc.c (os_obj_of, os_each_obj): hide objects to be finalized.
 	  [ruby-dev:31810]
 
+Wed Jun  4 19:16:40 2008  Yukihiro Matsumoto  <matz@r...>
+
+	* eval.c (remove_method): should not remove undef place holder.
+	  [ruby-dev:31817]
+
 Tue Jun  3 15:22:47 2008  Nobuyoshi Nakada  <nobu@r...>
 
 	* process.c (struct rb_exec_arg): proc should be a VALUE.
Index: ruby_1_8_6/version.h
===================================================================
--- ruby_1_8_6/version.h	(revision 16815)
+++ ruby_1_8_6/version.h	(revision 16816)
@@ -2,7 +2,7 @@
 #define RUBY_RELEASE_DATE "2008-06-04"
 #define RUBY_VERSION_CODE 186
 #define RUBY_RELEASE_CODE 20080604
-#define RUBY_PATCHLEVEL 124
+#define RUBY_PATCHLEVEL 125
 
 #define RUBY_VERSION_MAJOR 1
 #define RUBY_VERSION_MINOR 8
Index: ruby_1_8_6/eval.c
===================================================================
--- ruby_1_8_6/eval.c	(revision 16815)
+++ ruby_1_8_6/eval.c	(revision 16816)
@@ -569,8 +569,13 @@
     if (mid == __id__ || mid == __send__ || mid == init) {
 	rb_warn("removing `%s' may cause serious problem", rb_id2name(mid));
     }
-    if (!st_delete(RCLASS(klass)->m_tbl, &mid, (st_data_t *)&body) ||
-	!body->nd_body) {
+    if (st_lookup(RCLASS(klass)->m_tbl, mid, (st_data_t *)&body)) {
+	if (!body || !body->nd_body) body = 0;
+	else {
+	    st_delete(RCLASS(klass)->m_tbl, &mid, (st_data_t *)&body);
+	}
+    }
+    if (!body) {
 	rb_name_error(mid, "method `%s' not defined in %s",
 		      rb_id2name(mid), rb_class2name(klass));
     }

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/

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