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 2013/07/04 20:25:52 UTC

svn commit: r1499838 - in /commons/proper/math/trunk/src/test/java/org/apache/commons/math3: distribution/WeibullDistributionTest.java fitting/HarmonicFitterTest.java optimization/fitting/HarmonicFitterTest.java

Author: sebb
Date: Thu Jul  4 18:25:51 2013
New Revision: 1499838

URL: http://svn.apache.org/r1499838
Log:
Avoid Findbugs dead store warnings

Modified:
    commons/proper/math/trunk/src/test/java/org/apache/commons/math3/distribution/WeibullDistributionTest.java
    commons/proper/math/trunk/src/test/java/org/apache/commons/math3/fitting/HarmonicFitterTest.java
    commons/proper/math/trunk/src/test/java/org/apache/commons/math3/optimization/fitting/HarmonicFitterTest.java

Modified: commons/proper/math/trunk/src/test/java/org/apache/commons/math3/distribution/WeibullDistributionTest.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/test/java/org/apache/commons/math3/distribution/WeibullDistributionTest.java?rev=1499838&r1=1499837&r2=1499838&view=diff
==============================================================================
--- commons/proper/math/trunk/src/test/java/org/apache/commons/math3/distribution/WeibullDistributionTest.java (original)
+++ commons/proper/math/trunk/src/test/java/org/apache/commons/math3/distribution/WeibullDistributionTest.java Thu Jul  4 18:25:51 2013
@@ -76,7 +76,7 @@ public class WeibullDistributionTest ext
         WeibullDistribution dist = new WeibullDistribution(1, 2);
         Assert.assertEquals(1, dist.getShape(), 0);
         try {
-            dist = new WeibullDistribution(0, 2);
+            new WeibullDistribution(0, 2);
             Assert.fail("NotStrictlyPositiveException expected");
         } catch (NotStrictlyPositiveException e) {
             // Expected.
@@ -88,7 +88,7 @@ public class WeibullDistributionTest ext
         WeibullDistribution dist = new WeibullDistribution(1, 2);
         Assert.assertEquals(2, dist.getScale(), 0);
         try {
-            dist = new WeibullDistribution(1, 0);
+            new WeibullDistribution(1, 0);
             Assert.fail("NotStrictlyPositiveException expected");
         } catch (NotStrictlyPositiveException e) {
             // Expected.

Modified: commons/proper/math/trunk/src/test/java/org/apache/commons/math3/fitting/HarmonicFitterTest.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/test/java/org/apache/commons/math3/fitting/HarmonicFitterTest.java?rev=1499838&r1=1499837&r2=1499838&view=diff
==============================================================================
--- commons/proper/math/trunk/src/test/java/org/apache/commons/math3/fitting/HarmonicFitterTest.java (original)
+++ commons/proper/math/trunk/src/test/java/org/apache/commons/math3/fitting/HarmonicFitterTest.java Thu Jul  4 18:25:51 2013
@@ -179,7 +179,6 @@ public class HarmonicFitterTest {
         // and period 12, and all sample points are taken at integer abscissae
         // so function values all belong to the integer subset {-3, -2, -1, 0,
         // 1, 2, 3}.
-        @SuppressWarnings("unused")
-        final HarmonicFitter.ParameterGuesser guesser = new HarmonicFitter.ParameterGuesser(points);
+        new HarmonicFitter.ParameterGuesser(points);
     }
 }

Modified: commons/proper/math/trunk/src/test/java/org/apache/commons/math3/optimization/fitting/HarmonicFitterTest.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/test/java/org/apache/commons/math3/optimization/fitting/HarmonicFitterTest.java?rev=1499838&r1=1499837&r2=1499838&view=diff
==============================================================================
--- commons/proper/math/trunk/src/test/java/org/apache/commons/math3/optimization/fitting/HarmonicFitterTest.java (original)
+++ commons/proper/math/trunk/src/test/java/org/apache/commons/math3/optimization/fitting/HarmonicFitterTest.java Thu Jul  4 18:25:51 2013
@@ -196,7 +196,6 @@ public class HarmonicFitterTest {
         // and period 12, and all sample points are taken at integer abscissae
         // so function values all belong to the integer subset {-3, -2, -1, 0,
         // 1, 2, 3}.
-        final HarmonicFitter.ParameterGuesser guesser
-            = new HarmonicFitter.ParameterGuesser(points);
+        new HarmonicFitter.ParameterGuesser(points);
     }
 }