diff options
| -rw-r--r-- | ChangeLog | 5 | ||||
| -rw-r--r-- | range.c | 8 |
2 files changed, 9 insertions, 4 deletions
@@ -1,3 +1,8 @@ +Tue Feb 24 00:24:13 2009 Yukihiro Matsumoto <matz@ruby-lang.org> + + * range.c (range_each): check #succ only when it is really + needed. small performance improvement. + Tue Feb 24 00:19:33 2009 Yukihiro Matsumoto <matz@ruby-lang.org> * lib/pathname.rb (Pathname#foreachline): removed wrongly @@ -417,10 +417,6 @@ range_each(VALUE range) beg = RANGE_BEG(range); end = RANGE_END(range); - if (!rb_respond_to(beg, id_succ)) { - rb_raise(rb_eTypeError, "can't iterate from %s", - rb_obj_classname(beg)); - } if (FIXNUM_P(beg) && FIXNUM_P(end)) { /* fixnums are special */ long lim = FIX2LONG(end); long i; @@ -439,6 +435,10 @@ range_each(VALUE range) rb_block_call(beg, rb_intern("upto"), 2, args, rb_yield, 0); } else { + if (!rb_respond_to(beg, id_succ)) { + rb_raise(rb_eTypeError, "can't iterate from %s", + rb_obj_classname(beg)); + } range_each_func(range, each_i, NULL); } return range; |