You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2015/12/11 23:55:53 UTC

svn commit: r1719587 - /jmeter/trunk/src/core/org/apache/jmeter/report/processor/ErrorsSummaryConsumer.java

Author: pmouawad
Date: Fri Dec 11 22:55:52 2015
New Revision: 1719587

URL: http://svn.apache.org/viewvc?rev=1719587&view=rev
Log:
Remove autoboxing

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/report/processor/ErrorsSummaryConsumer.java

Modified: jmeter/trunk/src/core/org/apache/jmeter/report/processor/ErrorsSummaryConsumer.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/report/processor/ErrorsSummaryConsumer.java?rev=1719587&r1=1719586&r2=1719587&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/report/processor/ErrorsSummaryConsumer.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/report/processor/ErrorsSummaryConsumer.java Fri Dec 11 22:55:52 2015
@@ -33,6 +33,7 @@ import org.apache.jmeter.util.JMeterUtil
  */
 public class ErrorsSummaryConsumer extends AbstractSummaryConsumer<Long> {
 
+    private static final Long ZERO = Long.valueOf(0);
     private static final String ASSERTION_FAILED = "Assertion failed"; //$NON-NLS-1$
     private long errorCount = 0L;
 
@@ -56,9 +57,9 @@ public class ErrorsSummaryConsumer exten
         result.addResult(new ValueResultData(key != null ? key : JMeterUtils
                 .getResString("reportgenerator_summary_total")));
         result.addResult(new ValueResultData(data));
-        result.addResult(new ValueResultData((double) data * 100 / errorCount));
-        result.addResult(new ValueResultData((double) data * 100
-                / getOverallInfo().getData()));
+        result.addResult(new ValueResultData(Double.valueOf(((double) data.longValue() * 100 / errorCount))));
+        result.addResult(new ValueResultData(Double.valueOf((double) data.longValue() * 100
+                / getOverallInfo().getData().doubleValue())));
         return result;
     }
 
@@ -101,9 +102,9 @@ public class ErrorsSummaryConsumer exten
         SummaryInfo overallInfo = getOverallInfo();
         Long overallData = overallInfo.getData();
         if (overallData == null) {
-            overallData = Long.valueOf(0);
+            overallData = ZERO;
         }
-        overallInfo.setData(overallData + 1);
+        overallInfo.setData(Long.valueOf(overallData.longValue() + 1));
 
         // Process only failed samples
         if (!sample.getSuccess()) {
@@ -113,7 +114,7 @@ public class ErrorsSummaryConsumer exten
             if (data == null) {
                 data = Long.valueOf(1);
             }
-            info.setData(data + 1);
+            info.setData(Long.valueOf(data.longValue() + 1));
         }
     }