This patch makes sure FloatBuffer and DoubleBuffer objects compare
properly when they contain NaN values.  I've already checked in Mauve
tests cases.  Ok to commit?

AG



2005-12-06  Anthony Green  <[EMAIL PROTECTED]>

        * java/nio/DoubleBuffer.java (compareTo): Handle NaN values within
        the buffer.
        * java/nio/FloatBuffer.java (compareTo): Ditto.


Index: java/nio/FloatBuffer.java
===================================================================
RCS file: /cvsroot/classpath/classpath/java/nio/FloatBuffer.java,v
retrieving revision 1.20
diff -u -r1.20 FloatBuffer.java
--- java/nio/FloatBuffer.java   2 Jul 2005 20:32:39 -0000       1.20
+++ java/nio/FloatBuffer.java   6 Dec 2005 11:02:39 -0000
@@ -303,7 +303,11 @@
           
        if (a < b)
          return -1;
-          
+
+       // Handle the case where a and b are NaN.
+       if ((a != a) && (b != b))
+         continue;
+
        return 1;
       }
       
Index: java/nio/DoubleBuffer.java
===================================================================
RCS file: /cvsroot/classpath/classpath/java/nio/DoubleBuffer.java,v
retrieving revision 1.20
diff -u -r1.20 DoubleBuffer.java
--- java/nio/DoubleBuffer.java  2 Jul 2005 20:32:39 -0000       1.20
+++ java/nio/DoubleBuffer.java  6 Dec 2005 11:02:39 -0000
@@ -304,6 +304,10 @@
        if (a < b)
          return -1;
           
+       // Handle the case where a and b are NaN.
+       if ((a != a) && (b != b))
+         continue;
+
        return 1;
       }
       




_______________________________________________
Classpath-patches mailing list
Classpath-patches@gnu.org
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to