commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gilles <gil...@harfang.homelinux.org>
Subject Re: svn commit: r1517404 - /commons/proper/math/trunk/src/main/java/org/apache/commons/math3/util/FastMath.java
Date Mon, 26 Aug 2013 11:10:08 GMT
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


Mime
View raw message