diff options
author | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2009-06-03 00:02:14 +0000 |
---|---|---|
committer | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2009-06-03 00:02:14 +0000 |
commit | 967001d9da6a2f26736e092dba924748fa80d099 (patch) | |
tree | 1d6eb55d1fd81c31f83bacb03200dc812886944a /enum.c | |
parent | 6eaf79f39a92687bcdbe5b0e9bda2b639f1e8ede (diff) | |
download | ruby-967001d9da6a2f26736e092dba924748fa80d099.tar.gz |
* enum.c (first_i): Enumerator#first should consume only what is
needed. a patch from Marc-Andre Lafortune. [ruby-core:23661]
* enum.c (enum_first): call to_int once for an argument. based on
a patch from Marc-Andre Lafortune.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@23622 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'enum.c')
-rw-r--r-- | enum.c | 16 |
1 files changed, 10 insertions, 6 deletions
@@ -617,14 +617,14 @@ first_i(VALUE i, VALUE *ary, int argc, VALUE *argv) rb_iter_break(); } else { - long n = NUM2LONG(ary[0]); + long n = ary[0]; + rb_ary_push(ary[1], i); + n--; if (n <= 0) { rb_iter_break(); } - rb_ary_push(ary[1], i); - n--; - ary[0] = INT2NUM(n); + ary[0] = n; } return Qnil; } @@ -649,9 +649,13 @@ enum_first(int argc, VALUE *argv, VALUE obj) ary[0] = ary[1] = Qnil; } else { + long len; + rb_scan_args(argc, argv, "01", &n); - ary[0] = n; - ary[1] = rb_ary_new2(NUM2LONG(n)); + len = NUM2LONG(n); + if (len == 0) return rb_ary_new2(0); + ary[0] = len; + ary[1] = rb_ary_new2(len); } rb_block_call(obj, id_each, 0, 0, first_i, (VALUE)ary); |