You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@jmeter.apache.org by se...@apache.org on 2016/05/08 12:16:46 UTC

svn commit: r1742805 - /jmeter/trunk/src/core/org/apache/jmeter/report/processor/NormalizerSampleConsumer.java

Author: sebb
Date: Sun May  8 12:16:45 2016
New Revision: 1742805

URL: http://svn.apache.org/viewvc?rev=1742805&view=rev
Log:
Immutable private fields should be final

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

Modified: jmeter/trunk/src/core/org/apache/jmeter/report/processor/NormalizerSampleConsumer.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/report/processor/NormalizerSampleConsumer.java?rev=1742805&r1=1742804&r2=1742805&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/report/processor/NormalizerSampleConsumer.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/report/processor/NormalizerSampleConsumer.java Sun May  8 12:16:45 2016
@@ -37,11 +37,11 @@ public class NormalizerSampleConsumer ex
 
     private static final String PARSE_TIMESTAMP_EXCEPTION_MESSAGE = "Could not parse timeStamp <%s> on sample %s";
 
-    private static String DEFAULT_DATE_FORMAT = "HH:mm:ss";
+    private static final String DEFAULT_DATE_FORMAT = "HH:mm:ss";
 
     private int timestamp;
 
-    private SimpleDateFormat df = new SimpleDateFormat(
+    private final SimpleDateFormat df = new SimpleDateFormat(
             JMeterUtils.getPropDefault(
                     "jmeter.save.saveservice.timestamp_format",
                     DEFAULT_DATE_FORMAT));