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

[sebb]

Author: sebb
Date: Wed Jan 19 20:26:55 2011
New Revision: 1060967

URL: http://svn.apache.org/viewvc?rev=1060967&view=rev
Log:
MATH-486 FastMath toRadian and toDegree don't handle large double numbers well

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=1060967&r1=1060966&r2=1060967&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 20:26:55 2011
@@ -3239,7 +3239,10 @@ public class FastMath {
         final double facta = 0.01745329052209854;
         final double factb = 1.997844754509471E-9;
 
-        double temp = x * 1073741824.0;
+        double temp = 0;
+        if (abs(x) < Double.MAX_VALUE/1073741824.0) { // prevent overflow to 
infinity
+            temp = x * 1073741824.0;
+        }
         double xa = x + temp - temp;
         double xb = x - xa;
 
@@ -3264,7 +3267,10 @@ public class FastMath {
         final double facta = 57.2957763671875;
         final double factb = 3.145894820876798E-6;
 
-        double temp = x * 1073741824.0;
+        double temp = 0;
+        if (abs(x) < Double.MAX_VALUE/1073741824.0) { // prevent overflow to 
infinity
+            temp = x * 1073741824.0;
+        }
         double xa = x + temp - temp;
         double xb = x - xa;