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 2017/03/08 15:55:50 UTC

svn commit: r1785992 - /jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java

Author: sebb
Date: Wed Mar  8 15:55:50 2017
New Revision: 1785992

URL: http://svn.apache.org/viewvc?rev=1785992&view=rev
Log:
Can now simplify code further

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java

Modified: jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java?rev=1785992&r1=1785991&r2=1785992&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java Wed Mar  8 15:55:50 2017
@@ -323,14 +323,12 @@ public class ResultCollector extends Abs
             }
             instanceCount++;
             try {
-                String filename = getFilename();
-                if (filename != null) {
-                    if (out == null) {
-                        try {
-                            out = getFileWriter(filename, getSaveConfig());
-                        } catch (FileNotFoundException e) {
-                            out = null;
-                        }
+                if (out == null) {
+                    try {
+                        // Note: getFileWriter ignores a null filename
+                        out = getFileWriter(getFilename(), getSaveConfig());
+                    } catch (FileNotFoundException e) {
+                        out = null;
                     }
                 }
                 if (getVisualizer() != null) {