* array.c (rb_ary_product): Use tmpary instead, to ensure marking

arrays by GC.


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@28050 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
shyouhei 2010-05-28 08:51:41 +00:00
parent 97b1c9edd2
commit 11cef3cca3
2 changed files with 8 additions and 3 deletions

View File

@ -1,3 +1,8 @@
Fri May 28 17:34:48 2010 URABE Shyouhei <shyouhei@ruby-lang.org>
* array.c (rb_ary_product): Use tmpary instead, to ensure marking
arrays by GC.
Fri May 28 11:40:07 2010 URABE Shyouhei <shyouhei@ruby-lang.org>
* array.c (rb_ary_product): Do not rely on GC, t0 should be

View File

@ -4294,9 +4294,9 @@ static VALUE
rb_ary_product(int argc, VALUE *argv, VALUE ary)
{
int n = argc+1; /* How many arrays we're operating on */
volatile VALUE t0 = tmpbuf(n, sizeof(VALUE));
volatile VALUE t0 = tmpary(n);
volatile VALUE t1 = tmpbuf(n, sizeof(int));
VALUE *arrays = (VALUE*)RSTRING_PTR(t0); /* The arrays we're computing the product of */
VALUE *arrays = RARRAY_PTR(t0); /* The arrays we're computing the product of */
int *counters = (int*)RSTRING_PTR(t1); /* The current position in each one */
VALUE result = Qnil; /* The array we'll be returning, when no block given */
long i,j;
@ -4372,7 +4372,7 @@ rb_ary_product(int argc, VALUE *argv, VALUE ary)
}
}
done:
tmpbuf_discard(t0);
tmpary_discard(t0);
tmpbuf_discard(t1);
return NIL_P(result) ? ary : result;