You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by se...@apache.org on 2011/01/19 20:05:57 UTC

svn commit: r1060905 - /commons/proper/math/trunk/src/main/java/org/apache/commons/math/util/FastMath.java

Author: sebb
Date: Wed Jan 19 19:05:56 2011
New Revision: 1060905

URL: http://svn.apache.org/viewvc?rev=1060905&view=rev
Log:
MATH-479 FastMath.signum(-0.0) does not agree with Math.signum(-0.0)
(tests already added as part of math483)

Modified:
    commons/proper/math/trunk/src/main/java/org/apache/commons/math/util/FastMath.java

Modified: commons/proper/math/trunk/src/main/java/org/apache/commons/math/util/FastMath.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math/util/FastMath.java?rev=1060905&r1=1060904&r2=1060905&view=diff
==============================================================================
--- commons/proper/math/trunk/src/main/java/org/apache/commons/math/util/FastMath.java (original)
+++ commons/proper/math/trunk/src/main/java/org/apache/commons/math/util/FastMath.java Wed Jan 19 19:05:56 2011
@@ -579,7 +579,7 @@ public class FastMath {
      * @return -1, 0, +1 or NaN depending on sign of a
      */
     public static double signum(final double a) {
-        return (a < 0.0) ? -1.0 : ((a > 0.0) ? 1.0 : (Double.isNaN(a) ? Double.NaN : 0.0));
+       return (a < 0.0) ? -1.0 : ((a > 0.0) ? 1.0 : a);
     }
 
     /** Compute next number towards positive infinity.