diff options
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | array.c | 2 |
2 files changed, 6 insertions, 1 deletions
@@ -1,3 +1,8 @@ +Mon Sep 10 13:44:37 2007 Yukihiro Matsumoto <matz@ruby-lang.org> + + * array.c (rb_ary_cycle): avoid infinite loop for empty array. + based on a patch from David Flanagan. [ruby-core:12085] + Mon Sep 10 01:21:23 2007 Yukihiro Matsumoto <matz@ruby-lang.org> * marshal.c (r_object0): struct allocation first to check if a @@ -2939,7 +2939,7 @@ rb_ary_cycle(VALUE ary) long i; RETURN_ENUMERATOR(ary, 0, 0); - for (;;) { + while (RARRAY_LEN(ary) > 0) { for (i=0; i<RARRAY_LEN(ary); i++) { rb_yield(RARRAY_PTR(ary)[i]); } |