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

ruby-changes:22530

From: naruse <ko1@a...>
Date: Mon, 13 Feb 2012 13:53:21 +0900 (JST)
Subject: [ruby-changes:22530] naruse:r34579 (ruby_1_9_3): merge revision(s) 33935,33936,33987: [Backport #5702]

naruse	2012-02-13 13:51:21 +0900 (Mon, 13 Feb 2012)

  New Revision: 34579

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

  Log:
    merge revision(s) 33935,33936,33987: [Backport #5702]
    
    * variable.c (set_const_visibility): Module#private_constant has
      changed the visibility of only the first argument.  Now it changes
      all of them.  [ruby-list:48558]
    
    * test/ruby/test_module.rb: add a test for above.
    
    * variable.c (set_const_visibility): print a warning when no argument
      is passwd to Module#private_constant.  [ruby-list:48558]
    
    * vm_method.c (set_method_visibility): ditto for
      Module#private_class_method.
    
    * variable.c (set_const_visibility): clear inine-cache when constant's
      visibility is modified. [ruby-dev:44929]
    
    * test/ruby/test_module.rb (test_private_constants_clear_inlinecache):
      add test for it.

  Modified files:
    branches/ruby_1_9_3/ChangeLog
    branches/ruby_1_9_3/test/ruby/test_module.rb
    branches/ruby_1_9_3/variable.c
    branches/ruby_1_9_3/version.h
    branches/ruby_1_9_3/vm_method.c

Index: ruby_1_9_3/ChangeLog
===================================================================
--- ruby_1_9_3/ChangeLog	(revision 34578)
+++ ruby_1_9_3/ChangeLog	(revision 34579)
@@ -233,6 +233,30 @@
 
 	* test/ruby/test_module.rb: add a test for above.
 
+Sat Feb 11 02:39:09 2012  CHIKANAGA Tomoyuki  <nagachika00@g...>
+
+	* variable.c (set_const_visibility): clear inine-cache when constant's
+	  visibility is modified. [ruby-dev:44929]
+
+	* test/ruby/test_module.rb (test_private_constants_clear_inlinecache):
+	  add test for it.
+
+Sat Feb 11 02:39:09 2012  Yusuke Endoh  <mame@t...>
+
+	* variable.c (set_const_visibility): print a warning when no argument
+	  is passwd to Module#private_constant.  [ruby-list:48558]
+
+	* vm_method.c (set_method_visibility): ditto for
+	  Module#private_class_method.
+
+Sat Feb 11 02:39:09 2012  Yusuke Endoh  <mame@t...>
+
+	* variable.c (set_const_visibility): Module#private_constant has
+	  changed the visibility of only the first argument.  Now it changes
+	  all of them.  [ruby-list:48558]
+
+	* test/ruby/test_module.rb: add a test for above.
+
 Sat Feb 11 02:26:51 2012  Martin Bosslet  <Martin.Bosslet@g...>
 
 	* lib/openssl/buffering.rb: Force multi-byte strings to be treated as
Index: ruby_1_9_3/variable.c
===================================================================
--- ruby_1_9_3/variable.c	(revision 34578)
+++ ruby_1_9_3/variable.c	(revision 34579)
@@ -1952,13 +1952,22 @@
 		 "Insecure: can't change constant visibility");
     }
 
+    if (argc == 0) {
+	rb_warning("%s with no argument is just ignored", rb_id2name(rb_frame_callee()));
+    }
+
     for (i = 0; i < argc; i++) {
-	id = rb_to_id(argv[i]);
-	if (RCLASS_CONST_TBL(mod) && st_lookup(RCLASS_CONST_TBL(mod), (st_data_t)id, &v)) {
+	VALUE val = argv[i];
+	id = rb_to_id(val);
+	if (RCLASS_CONST_TBL(mod) &&
+	    st_lookup(RCLASS_CONST_TBL(mod), (st_data_t)id, &v)) {
 	    ((rb_const_entry_t*)v)->flag = flag;
-	    return;
 	}
-	rb_name_error(id, "constant %s::%s not defined", rb_class2name(mod), rb_id2name(id));
+	else {
+	    if ( i > 0 )
+		rb_clear_cache_by_class(mod);
+	    rb_name_error(id, "constant %s::%s not defined", rb_class2name(mod), rb_id2name(id));
+	}
     }
     rb_clear_cache_by_class(mod);
 }
Index: ruby_1_9_3/vm_method.c
===================================================================
--- ruby_1_9_3/vm_method.c	(revision 34578)
+++ ruby_1_9_3/vm_method.c	(revision 34579)
@@ -963,6 +963,11 @@
 {
     int i;
     secure_visibility(self);
+
+    if (argc == 0) {
+	rb_warning("%s with no argument is just ignored", rb_id2name(rb_frame_callee()));
+    }
+
     for (i = 0; i < argc; i++) {
 	rb_export_method(self, rb_to_id(argv[i]), ex);
     }
Index: ruby_1_9_3/version.h
===================================================================
--- ruby_1_9_3/version.h	(revision 34578)
+++ ruby_1_9_3/version.h	(revision 34579)
@@ -1,5 +1,5 @@
 #define RUBY_VERSION "1.9.3"
-#define RUBY_PATCHLEVEL 104
+#define RUBY_PATCHLEVEL 105
 
 #define RUBY_RELEASE_DATE "2012-02-13"
 #define RUBY_RELEASE_YEAR 2012
Index: ruby_1_9_3/test/ruby/test_module.rb
===================================================================
--- ruby_1_9_3/test/ruby/test_module.rb	(revision 34578)
+++ ruby_1_9_3/test/ruby/test_module.rb	(revision 34579)
@@ -1077,6 +1077,19 @@
     assert_raise(NameError) { c::FOO }
   end
 
+  def test_private_constant2
+    c = Class.new
+    c.const_set(:FOO, "foo")
+    c.const_set(:BAR, "bar")
+    assert_equal("foo", c::FOO)
+    assert_equal("bar", c::BAR)
+    c.private_constant(:FOO, :BAR)
+    assert_raise(NameError) { c::FOO }
+    assert_raise(NameError) { c::BAR }
+    assert_equal("foo", c.class_eval("FOO"))
+    assert_equal("bar", c.class_eval("BAR"))
+  end
+
   class PrivateClass
   end
   private_constant :PrivateClass
@@ -1124,6 +1137,27 @@
     assert_in_out_err([], src, %w(Object :ok), [])
   end
 
+  def test_private_constants_clear_inlinecache
+    bug5702 = '[ruby-dev:44929]'
+    src = <<-INPUT
+    class A
+      C = :Const
+      def self.get_C
+        A::C
+      end
+      # fill cache
+      A.get_C
+      private_constant :C, :D rescue nil
+      begin
+        A.get_C
+      rescue NameError
+        puts "A.get_C"
+      end
+    end
+    INPUT
+    assert_in_out_err([], src, %w(A.get_C), [], bug5702)
+  end
+
   def test_constant_lookup_in_method_defined_by_class_eval
     src = <<-INPUT
       class A

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

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