matz	2009-02-24 00:46:02 +0900 (Tue, 24 Feb 2009)

  New Revision: 22574

  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=22574

  Log:
    * range.c (range_each): check #succ only when it is really
      needed.  small performance improvement.

  Modified files:
    trunk/ChangeLog
    trunk/range.c

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 22573)
+++ ChangeLog	(revision 22574)
@@ -1,3 +1,8 @@
+Tue Feb 24 00:24:13 2009  Yukihiro Matsumoto  <matz@r...>
+
+	* 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@r...>
 
 	* lib/pathname.rb (Pathname#foreachline): removed wrongly
Index: range.c
===================================================================
--- range.c	(revision 22573)
+++ range.c	(revision 22574)
@@ -417,10 +417,6 @@
     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 @@
 	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;

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