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

ruby-changes:28871

From: nobu <ko1@a...>
Date: Sat, 25 May 2013 11:29:02 +0900 (JST)
Subject: [ruby-changes:28871] nobu:r40923 (trunk): no clearing method cache

nobu	2013-05-25 11:28:51 +0900 (Sat, 25 May 2013)

  New Revision: 40923

  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=40923

  Log:
    no clearing method cache
    
    * variable.c (set_const_visibility): return without clearing method
      cache if no arguments.
    * vm_method.c (set_method_visibility): ditto.

  Modified files:
    trunk/ChangeLog
    trunk/variable.c
    trunk/vm_method.c

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 40922)
+++ ChangeLog	(revision 40923)
@@ -1,3 +1,10 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1
+Sat May 25 11:28:49 2013  Nobuyoshi Nakada  <nobu@r...>
+
+	* variable.c (set_const_visibility): return without clearing method
+	  cache if no arguments.
+
+	* vm_method.c (set_method_visibility): ditto.
+
 Sat May 25 11:27:32 2013  Nobuyoshi Nakada  <nobu@r...>
 
 	* vm_method.c (set_method_visibility): quote unprintable method name.
Index: variable.c
===================================================================
--- variable.c	(revision 40922)
+++ variable.c	(revision 40923)
@@ -2232,6 +2232,7 @@ set_const_visibility(VALUE mod, int argc https://github.com/ruby/ruby/blob/trunk/variable.c#L2232
     if (argc == 0) {
 	rb_warning("%"PRIsVALUE" with no argument is just ignored",
 		   QUOTE_ID(rb_frame_callee()));
+	return;
     }
 
     for (i = 0; i < argc; i++) {
Index: vm_method.c
===================================================================
--- vm_method.c	(revision 40922)
+++ vm_method.c	(revision 40923)
@@ -1276,6 +1276,7 @@ set_method_visibility(VALUE self, int ar https://github.com/ruby/ruby/blob/trunk/vm_method.c#L1276
     if (argc == 0) {
 	rb_warning("%"PRIsVALUE" with no argument is just ignored",
 		   QUOTE_ID(rb_frame_callee()));
+	return;
     }
 
     for (i = 0; i < argc; i++) {

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

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