nobu	2008-05-22 12:40:57 +0900 (Thu, 22 May 2008)

  New Revision: 16522

  Modified files:
    trunk/ChangeLog
    trunk/array.c
    trunk/test/ruby/test_array.rb

  Log:
    * array.c (flatten): check if reentered.  [ruby-dev:34798]


  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/trunk/array.c?r1=16522&r2=16521&diff_format=u
  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/trunk/ChangeLog?r1=16522&r2=16521&diff_format=u
  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/trunk/test/ruby/test_array.rb?r1=16522&r2=16521&diff_format=u

Index: array.c
===================================================================
--- array.c	(revision 16521)
+++ array.c	(revision 16522)
@@ -2831,8 +2831,8 @@
     st_table *memo;
     st_data_t id;
 
-    stack = rb_ary_new();
-    result = ary_new(rb_class_of(ary), RARRAY_LEN(ary));
+    stack = ary_new(0, ARY_DEFAULT_SIZE);
+    result = ary_new(0, RARRAY_LEN(ary));
     memo = st_init_numtable();
     st_insert(memo, (st_data_t)ary, (st_data_t)Qtrue);
     *modified = 0;
@@ -2841,6 +2841,9 @@
 	while (i < RARRAY_LEN(ary)) {
 	    elt = RARRAY_PTR(ary)[i++];
 	    tmp = rb_check_array_type(elt);
+	    if (RBASIC(result)->klass) {
+		rb_raise(rb_eRuntimeError, "flatten reentered");
+	    }
 	    if (NIL_P(tmp) || (level >= 0 && RARRAY_LEN(stack) / 2 >= level)) {
 		rb_ary_push(result, elt);
 	    }
@@ -2870,6 +2873,7 @@
 
     st_free_table(memo);
 
+    RBASIC(result)->klass = rb_class_of(ary);
     return result;
 }
 
Index: ChangeLog
===================================================================
--- ChangeLog	(revision 16521)
+++ ChangeLog	(revision 16522)
@@ -1,3 +1,7 @@
+Thu May 22 12:40:54 2008  Nobuyoshi Nakada  <nobu@r...>
+
+	* array.c (flatten): check if reentered.  [ruby-dev:34798]
+
 Thu May 22 11:39:59 2008  Tanaka Akira  <akr@f...>
 
 	* test/ruby/envutil.rb (assert_normal_exit): capture stdout and stderr
Index: test/ruby/test_array.rb
===================================================================
--- test/ruby/test_array.rb	(revision 16521)
+++ test/ruby/test_array.rb	(revision 16522)
@@ -726,6 +726,20 @@
                  @cls[@cls[@cls[@cls[],@cls[]],@cls[@cls[]],@cls[]],@cls[@cls[@cls[]]]].flatten)
   end
 
+  def test_flatten_with_callcc
+    respond_to?(:callcc) or require 'continuation'
+    o = Object.new
+    def o.to_ary() callcc {|k| @cont = k; [1,2,3]} end
+    begin
+      assert_equal([10, 20, 1, 2, 3, 30, 1, 2, 3, 40], [10, 20, o, 30, o, 40].flatten)
+    rescue => e
+    else
+      o.instance_eval {@cont}.call
+    end
+    assert_instance_of(RuntimeError, e, '[ruby-dev:34798]')
+    assert_match(/reentered/, e.message, '[ruby-dev:34798]')
+  end
+
   def test_hash
     a1 = @cls[ 'cat', 'dog' ]
     a2 = @cls[ 'cat', 'dog' ]

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