Re: [cp-patches] FYI: Define isnan explicitly in fdlibm

[Christian Thalinger]

On Sat, 2005-12-17 at 21:43 +0100, Guilhem Lavaux wrote:
> Hi,
> 
> Here is a patch to define isnan explicitly as a macro if it is not 
> already defined. At the same it removes the symbol renaming in namespace.h.

Hi!

As i merged the changes into CACAO, i noticed that this patch breaks
some stuff on mips-irix.  The fptest of the CACAO regression tests does
not pass anymore:

--- fptest.output       Tue Jul 19 10:52:11 2005
+++ Standard Input      Wed Jan 11 01:18:17 2006
@@ -14,10 +14,10 @@
   12.    + / 0 = +INF double: Infinity
   13.    - / 0 = -INF double: -Infinity
   14.    ------------------- test conversions
-  15.    convert NaNQ (NaN,NaN) to int:    0
-  16.    convert NaNQ (NaN,NaN) to int:    0
-  17.    convert NaNQ (NaN,NaN) to long:   0
-  18.    convert NaNQ (NaN,NaN) to long:   0
+  15.    convert NaNQ (NaN,NaN) to int:    2147483647
+  16.    convert NaNQ (NaN,NaN) to int:    2147483647
+  17.    convert NaNQ (NaN,NaN) to long:   9223372036854775807
+  18.    convert NaNQ (NaN,NaN) to long:   9223372036854775807
   19.    convert +INF (Infinity,Infinity) to int:    2147483647
   20.    convert +INF (Infinity,Infinity) to int:    2147483647
   21.    convert +INF (Infinity,Infinity) to long:   9223372036854775807

I didn't notice that earlier since i'm not using classpath cvs head on
that machine.  What was the reason for this patch?

TWISTI