ruby-changes:10952
From: shyouhei <ko1@a...>
Date: Sun, 22 Feb 2009 21:43:50 +0900 (JST)
Subject: [ruby-changes:10952] Ruby:r22528 (ruby_1_8_7): merge revision(s) 21298:
shyouhei 2009-02-22 21:43:42 +0900 (Sun, 22 Feb 2009) New Revision: 22528 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=22528 Log: merge revision(s) 21298: * numeric.c (ruby_float_step): extracted from num_step(). * range.c (range_step): uses ruby_float_step() for float range. [ruby-dev:37691] Modified files: branches/ruby_1_8_7/ChangeLog branches/ruby_1_8_7/numeric.c branches/ruby_1_8_7/range.c branches/ruby_1_8_7/version.h Index: ruby_1_8_7/numeric.c =================================================================== --- ruby_1_8_7/numeric.c (revision 22527) +++ ruby_1_8_7/numeric.c (revision 22528) @@ -1427,6 +1427,33 @@ } +int ruby_float_step _((VALUE from, VALUE to, VALUE step, int excl)); + +int +ruby_float_step(from, to, step, excl) + VALUE from, to, step; + int excl; +{ + if (TYPE(from) == T_FLOAT || TYPE(to) == T_FLOAT || TYPE(step) == T_FLOAT) { + const double epsilon = DBL_EPSILON; + double beg = NUM2DBL(from); + double end = NUM2DBL(to); + double unit = NUM2DBL(step); + double n = (end - beg)/unit; + double err = (fabs(beg) + fabs(end) + fabs(end-beg)) / fabs(unit) * epsilon; + long i; + + if (err>0.5) err=0.5; + n = floor(n + err); + if (!excl) n++; + for (i=0; i<n; i++) { + rb_yield(rb_float_new(i*unit+beg)); + } + return Qtrue; + } + return Qfalse; +} + /* * call-seq: * num.step(limit, step ) {|i| block } => num @@ -1501,22 +1528,7 @@ } } } - else if (TYPE(from) == T_FLOAT || TYPE(to) == T_FLOAT || TYPE(step) == T_FLOAT) { - const double epsilon = DBL_EPSILON; - double beg = NUM2DBL(from); - double end = NUM2DBL(to); - double unit = NUM2DBL(step); - double n = (end - beg)/unit; - double err = (fabs(beg) + fabs(end) + fabs(end-beg)) / fabs(unit) * epsilon; - long i; - - if (err>0.5) err=0.5; - n = floor(n + err) + 1; - for (i=0; i<n; i++) { - rb_yield(rb_float_new(i*unit+beg)); - } - } - else { + else if (!ruby_float_step(from, to, step, Qfalse)) { VALUE i = from; ID cmp; Index: ruby_1_8_7/ChangeLog =================================================================== --- ruby_1_8_7/ChangeLog (revision 22527) +++ ruby_1_8_7/ChangeLog (revision 22528) @@ -1,3 +1,10 @@ +Sun Feb 22 21:43:34 2009 Nobuyoshi Nakada <nobu@r...> + + * numeric.c (ruby_float_step): extracted from num_step(). + + * range.c (range_step): uses ruby_float_step() for float range. + [ruby-dev:37691] + Sun Feb 22 00:49:36 2009 Nobuyoshi Nakada <nobu@r...> * ext/extmk.rb (extmake): does not use both of makefile.rb and Index: ruby_1_8_7/version.h =================================================================== --- ruby_1_8_7/version.h (revision 22527) +++ ruby_1_8_7/version.h (revision 22528) @@ -2,7 +2,7 @@ #define RUBY_RELEASE_DATE "2009-02-22" #define RUBY_VERSION_CODE 187 #define RUBY_RELEASE_CODE 20090222 -#define RUBY_PATCHLEVEL 136 +#define RUBY_PATCHLEVEL 137 #define RUBY_VERSION_MAJOR 1 #define RUBY_VERSION_MINOR 8 Index: ruby_1_8_7/range.c =================================================================== --- ruby_1_8_7/range.c (revision 22527) +++ ruby_1_8_7/range.c (revision 22528) @@ -273,6 +273,8 @@ return Qnil; } +extern int ruby_float_step _((VALUE from, VALUE to, VALUE step, int excl)); + /* * call-seq: * rng.step(n=1) {| obj | block } => rng @@ -343,6 +345,9 @@ } } + else if (ruby_float_step(b, e, step, EXCL(range))) { + /* done */ + } else if (rb_obj_is_kind_of(b, rb_cNumeric) || !NIL_P(rb_check_to_integer(b, "to_int")) || !NIL_P(rb_check_to_integer(e, "to_int"))) { -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/