You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by db...@apache.org on 2013/08/26 03:05:12 UTC

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

Author: dbrosius
Date: Mon Aug 26 01:05:11 2013
New Revision: 1517404

URL: http://svn.apache.org/r1517404
Log:
remove dead store to 'resultb'

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

Modified: commons/proper/math/trunk/src/main/java/org/apache/commons/math3/util/FastMath.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math3/util/FastMath.java?rev=1517404&r1=1517403&r2=1517404&view=diff
==============================================================================
--- commons/proper/math/trunk/src/main/java/org/apache/commons/math3/util/FastMath.java (original)
+++ commons/proper/math/trunk/src/main/java/org/apache/commons/math3/util/FastMath.java Mon Aug 26 01:05:11 2013
@@ -2561,7 +2561,6 @@ public class FastMath {
             zb += pib - resultb;
 
             result = za + zb;
-            resultb = -(result - za - zb);
         }
 
 



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

Posted by Gilles <gi...@harfang.homelinux.org>.
On Mon, 26 Aug 2013 01:05:12 -0000, dbrosius@apache.org wrote:
> Author: dbrosius
> Date: Mon Aug 26 01:05:11 2013
> New Revision: 1517404
>
> URL: http://svn.apache.org/r1517404
> Log:
> remove dead store to 'resultb'
>
> Modified:
>
> 
> commons/proper/math/trunk/src/main/java/org/apache/commons/math3/util/FastMath.java
>
> Modified:
> 
> commons/proper/math/trunk/src/main/java/org/apache/commons/math3/util/FastMath.java
> URL:
> 
> http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math3/util/FastMath.java?rev=1517404&r1=1517403&r2=1517404&view=diff
> 
> ==============================================================================
> ---
> 
> commons/proper/math/trunk/src/main/java/org/apache/commons/math3/util/FastMath.java
> (original)
> +++
> 
> commons/proper/math/trunk/src/main/java/org/apache/commons/math3/util/FastMath.java
> Mon Aug 26 01:05:11 2013
> @@ -2561,7 +2561,6 @@ public class FastMath {
>              zb += pib - resultb;
>
>              result = za + zb;
> -            resultb = -(result - za - zb);
>          }

If I'm not mistaken, the declaration of "resultb" can be moved to 
inside
the block (and made "final").

Gilles


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org