From c691d55bcdd09081bb18d7ee88983175470f7249 Mon Sep 17 00:00:00 2001 From: usa Date: Fri, 27 Feb 2009 05:42:06 +0000 Subject: [PATCH] * numeric.c (flo_eq, flo_gt, flo_ge, flo_lt, flo_le, flo_eql): revert isnan() checks for VC++6. * insns.def (opt_lt, opt_gt): ditto. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@22654 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 7 +++++++ insns.def | 16 ++++++++++++++-- numeric.c | 37 ++++++++++++++++++++++++++++++++++++- 3 files changed, 57 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index 222cb1cf17..cb85f1237a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +Fri Feb 27 14:39:40 2009 NAKAMURA Usaku + + * numeric.c (flo_eq, flo_gt, flo_ge, flo_lt, flo_le, flo_eql): revert + isnan() checks for VC++6. + + * insns.def (opt_lt, opt_gt): ditto. + Fri Feb 27 14:23:09 2009 Nobuyoshi Nakada * lib/ostruct.rb (modifiable): check if really frozen. diff --git a/insns.def b/insns.def index 78318940f6..f82c5db326 100644 --- a/insns.def +++ b/insns.def @@ -1676,7 +1676,13 @@ opt_lt else if (HEAP_CLASS_OF(recv) == rb_cFloat && HEAP_CLASS_OF(obj) == rb_cFloat && BASIC_OP_UNREDEFINED_P(BOP_LT)) { - val = RFLOAT_VALUE(recv) < RFLOAT_VALUE(obj) ? Qtrue : Qfalse; + double a = RFLOAT_VALUE(recv); + double b = RFLOAT_VALUE(obj); +#if defined(_MSC_VER) && _MSC_VER < 1300 + if (isnan(a) || isnan(b)) val = Qfalse; + else +#endif + val = a < b ? Qtrue : Qfalse; } #endif else { @@ -1750,7 +1756,13 @@ opt_gt else if (HEAP_CLASS_OF(recv) == rb_cFloat && HEAP_CLASS_OF(obj) == rb_cFloat && BASIC_OP_UNREDEFINED_P(BOP_GT)) { - val = RFLOAT_VALUE(recv) > RFLOAT_VALUE(obj) ? Qtrue : Qfalse; + double a = RFLOAT_VALUE(recv); + double b = RFLOAT_VALUE(obj); +#if defined(_MSC_VER) && _MSC_VER < 1300 + if (isnan(a) || isnan(b)) val = Qfalse; + else +#endif + val = a > b ? Qtrue : Qfalse; } #endif else { diff --git a/numeric.c b/numeric.c index f987610362..2d6691de61 100644 --- a/numeric.c +++ b/numeric.c @@ -874,11 +874,17 @@ flo_eq(VALUE x, VALUE y) break; case T_FLOAT: b = RFLOAT_VALUE(y); +#if defined(_MSC_VER) && _MSC_VER < 1300 + if (isnan(b)) return Qfalse; +#endif break; default: return num_equal(x, y); } a = RFLOAT_VALUE(x); +#if defined(_MSC_VER) && _MSC_VER < 1300 + if (isnan(a)) return Qfalse; +#endif return (a == b)?Qtrue:Qfalse; } @@ -968,11 +974,17 @@ flo_gt(VALUE x, VALUE y) case T_FLOAT: b = RFLOAT_VALUE(y); +#if defined(_MSC_VER) && _MSC_VER < 1300 + if (isnan(b)) return Qfalse; +#endif break; default: return rb_num_coerce_relop(x, y, '>'); } +#if defined(_MSC_VER) && _MSC_VER < 1300 + if (isnan(a)) return Qfalse; +#endif return (a > b)?Qtrue:Qfalse; } @@ -1001,11 +1013,17 @@ flo_ge(VALUE x, VALUE y) case T_FLOAT: b = RFLOAT_VALUE(y); +#if defined(_MSC_VER) && _MSC_VER < 1300 + if (isnan(b)) return Qfalse; +#endif break; default: return rb_num_coerce_relop(x, y, rb_intern(">=")); } +#if defined(_MSC_VER) && _MSC_VER < 1300 + if (isnan(a)) return Qfalse; +#endif return (a >= b)?Qtrue:Qfalse; } @@ -1033,11 +1051,17 @@ flo_lt(VALUE x, VALUE y) case T_FLOAT: b = RFLOAT_VALUE(y); +#if defined(_MSC_VER) && _MSC_VER < 1300 + if (isnan(b)) return Qfalse; +#endif break; default: return rb_num_coerce_relop(x, y, '<'); } +#if defined(_MSC_VER) && _MSC_VER < 1300 + if (isnan(a)) return Qfalse; +#endif return (a < b)?Qtrue:Qfalse; } @@ -1066,11 +1090,17 @@ flo_le(VALUE x, VALUE y) case T_FLOAT: b = RFLOAT_VALUE(y); +#if defined(_MSC_VER) && _MSC_VER < 1300 + if (isnan(b)) return Qfalse; +#endif break; default: return rb_num_coerce_relop(x, y, rb_intern("<=")); } +#if defined(_MSC_VER) && _MSC_VER < 1300 + if (isnan(a)) return Qfalse; +#endif return (a <= b)?Qtrue:Qfalse; } @@ -1089,7 +1119,12 @@ static VALUE flo_eql(VALUE x, VALUE y) { if (TYPE(y) == T_FLOAT) { - if (RFLOAT_VALUE(x) == RFLOAT_VALUE(y)) + double a = RFLOAT_VALUE(x); + double b = RFLOAT_VALUE(y); +#if defined(_MSC_VER) && _MSC_VER < 1300 + if (isnan(a) || isnan(b)) return Qfalse; +#endif + if (a == b) return Qtrue; } return Qfalse;