nagachika	2014-02-15 15:53:01 +0900 (Sat, 15 Feb 2014)

  New Revision: 44967

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

  Log:
    merge revision(s) r44931: [Backport #9452]
    
    * vm_insnhelper.c (vm_call_method): should check ci->me->flag of
      a refining method in case the method is private.
      [ruby-core:60111] [Bug #9452]
    
    * vm_method.c (make_method_entry_refined): set me->flag of a refined
      method entry to NOEX_PUBLIC in case the original method is private
      and it is refined as a public method.  The original flag is stored
      in me->def->body.orig_me, so it's OK to make a refined method
      entry public.  [ruby-core:60111] [Bug #9452]
    
    * test/ruby/test_refinement.rb: related tests.

  Modified directories:
    branches/ruby_2_0_0/
  Modified files:
    branches/ruby_2_0_0/ChangeLog
    branches/ruby_2_0_0/test/ruby/test_refinement.rb
    branches/ruby_2_0_0/version.h
    branches/ruby_2_0_0/vm_insnhelper.c
    branches/ruby_2_0_0/vm_method.c
Index: ruby_2_0_0/ChangeLog
===================================================================
--- ruby_2_0_0/ChangeLog	(revision 44966)
+++ ruby_2_0_0/ChangeLog	(revision 44967)
@@ -1,3 +1,17 @@ https://github.com/ruby/ruby/blob/trunk/ruby_2_0_0/ChangeLog#L1
+Sat Feb 15 15:32:46 2014  Shugo Maeda  <shugo@r...>
+
+	* vm_insnhelper.c (vm_call_method): should check ci->me->flag of
+	  a refining method in case the method is private.
+	  [ruby-core:60111] [Bug #9452]
+
+	* vm_method.c (make_method_entry_refined): set me->flag of a refined
+	  method entry to NOEX_PUBLIC in case the original method is private
+	  and it is refined as a public method.  The original flag is stored
+	  in me->def->body.orig_me, so it's OK to make a refined method
+	  entry public.  [ruby-core:60111] [Bug #9452]
+
+	* test/ruby/test_refinement.rb: related tests.
+
 Tue Feb 11 23:21:02 2014  CHIKANAGA Tomoyuki  <nagachika@r...>
 
 	* test/ruby/test_struct.rb (test_struct_question_mark): add a testcase
Index: ruby_2_0_0/vm_method.c
===================================================================
--- ruby_2_0_0/vm_method.c	(revision 44966)
+++ ruby_2_0_0/vm_method.c	(revision 44967)
@@ -203,6 +203,7 @@ make_method_entry_refined(rb_method_entr https://github.com/ruby/ruby/blob/trunk/ruby_2_0_0/vm_method.c#L203
     *new_def->body.orig_me = *me;
     rb_vm_check_redefinition_opt_method(me, me->klass);
     if (me->def) me->def->alias_count++;
+    me->flag = NOEX_WITH_SAFE(NOEX_PUBLIC);
     me->def = new_def;
 }
 
Index: ruby_2_0_0/version.h
===================================================================
--- ruby_2_0_0/version.h	(revision 44966)
+++ ruby_2_0_0/version.h	(revision 44967)
@@ -1,10 +1,10 @@ https://github.com/ruby/ruby/blob/trunk/ruby_2_0_0/version.h#L1
 #define RUBY_VERSION "2.0.0"
-#define RUBY_RELEASE_DATE "2014-02-11"
-#define RUBY_PATCHLEVEL 402
+#define RUBY_RELEASE_DATE "2014-02-15"
+#define RUBY_PATCHLEVEL 403
 
 #define RUBY_RELEASE_YEAR 2014
 #define RUBY_RELEASE_MONTH 2
-#define RUBY_RELEASE_DAY 11
+#define RUBY_RELEASE_DAY 15
 
 #include "ruby/version.h"
 
Index: ruby_2_0_0/vm_insnhelper.c
===================================================================
--- ruby_2_0_0/vm_insnhelper.c	(revision 44966)
+++ ruby_2_0_0/vm_insnhelper.c	(revision 44967)
@@ -1839,7 +1839,7 @@ vm_call_method(rb_thread_t *th, rb_contr https://github.com/ruby/ruby/blob/trunk/ruby_2_0_0/vm_insnhelper.c#L1839
 		    ci->me = me;
 		    ci->defined_class = defined_class;
 		    if (me->def->type != VM_METHOD_TYPE_REFINED) {
-			goto normal_method_dispatch;
+			goto start_method_dispatch;
 		    }
 		}
 
@@ -1848,11 +1848,8 @@ vm_call_method(rb_thread_t *th, rb_contr https://github.com/ruby/ruby/blob/trunk/ruby_2_0_0/vm_insnhelper.c#L1848
 		    ci->me = ci->me->def->body.orig_me;
 		    if (UNDEFINED_METHOD_ENTRY_P(ci->me)) {
 			ci->me = 0;
-			goto start_method_dispatch;
-		    }
-		    else {
-			goto normal_method_dispatch;
 		    }
+		    goto start_method_dispatch;
 		}
 		else {
 		    klass = ci->me->klass;
Index: ruby_2_0_0/test/ruby/test_refinement.rb
===================================================================
--- ruby_2_0_0/test/ruby/test_refinement.rb	(revision 44966)
+++ ruby_2_0_0/test/ruby/test_refinement.rb	(revision 44967)
@@ -955,6 +955,53 @@ class TestRefinement < Test::Unit::TestC https://github.com/ruby/ruby/blob/trunk/ruby_2_0_0/test/ruby/test_refinement.rb#L955
     INPUT
   end
 
+  def test_adding_private_method
+    bug9452 = '[ruby-core:60111] [Bug #9452]'
+
+    assert_in_out_err([], <<-INPUT, ["Success!", "NoMethodError"], [], bug9452)
+      $VERBOSE = nil #to suppress warning "Refinements are experimental, ..."
+      module R
+        refine Object do
+          def m
+            puts "Success!"
+          end
+
+          private(:m)
+        end
+      end
+
+      using R
+
+      m
+      42.m rescue p($!.class)
+    INPUT
+  end
+
+  def test_making_private_method_public
+    bug9452 = '[ruby-core:60111] [Bug #9452]'
+
+    assert_in_out_err([], <<-INPUT, ["Success!", "Success!"], [], bug9452)
+        $VERBOSE = nil #to suppress warning "Refinements are experimental, ..."
+        class Object
+          private
+          def m
+          end
+        end
+
+        module R
+          refine Object do
+            def m
+              puts "Success!"
+            end
+          end
+        end
+
+        using R
+        m
+        42.m
+    INPUT
+  end
+
   private
 
   def eval_using(mod, s)

Property changes on: ruby_2_0_0
___________________________________________________________________
Modified: svn:mergeinfo
   Merged /trunk:r44931


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