Author: pmouawad
Date: Wed Nov 30 22:16:35 2011
New Revision: 1208836
URL: http://svn.apache.org/viewvc?rev=1208836&view=rev
Log:
Made code cleaner
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=1208836&r1=1208835&r2=1208836&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java Wed Nov 30 22:16:35
2011
@@ -389,10 +389,10 @@ public class ResultCollector extends Abs
// Find the name of the directory containing the file
// and create it - if there is one
File pdir = new File(filename).getParentFile();
- if (pdir != null) {
- pdir.mkdirs();// returns false if directory already exists, so need to check
again
- if (!pdir.exists()){
- log.warn("Error creating directories for "+pdir.toString());
+ if (pdir != null && !pdir.exists()) {
+ boolean mkdirResult = pdir.mkdirs();
+ if (!mkdirResult){
+ log.warn("Error creating directories for "+pdir.getAbsolutePath());
}
}
writer = new PrintWriter(new OutputStreamWriter(new BufferedOutputStream(new
FileOutputStream(filename,
|