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 2019/04/13 11:21:29 UTC
svn commit: r1857470 -
/jmeter/trunk/src/functions/org/apache/jmeter/functions/StringToFile.java
Author: pmouawad
Date: Sat Apr 13 11:21:28 2019
New Revision: 1857470
URL: http://svn.apache.org/viewvc?rev=1857470&view=rev
Log:
Drop weird char
Add NOSONAR for handled exceptions
Modified:
jmeter/trunk/src/functions/org/apache/jmeter/functions/StringToFile.java
Modified: jmeter/trunk/src/functions/org/apache/jmeter/functions/StringToFile.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/functions/org/apache/jmeter/functions/StringToFile.java?rev=1857470&r1=1857469&r2=1857470&view=diff
==============================================================================
--- jmeter/trunk/src/functions/org/apache/jmeter/functions/StringToFile.java (original)
+++ jmeter/trunk/src/functions/org/apache/jmeter/functions/StringToFile.java Sat Apr 13 11:21:28 2019
@@ -92,7 +92,7 @@ public class StringToFile extends Abstra
log.error("File name '{}' is empty", fileName);
return false;
}
- log.debug("Writing {}Â to file {} with charset {} and append {}", content, fileName, charset, append);
+ log.debug("Writing {} to file {} with charset {} and append {}", content, fileName, charset, append);
Lock localLock = new ReentrantLock();
Lock lock = lockMap.putIfAbsent(fileName, localLock);
@@ -107,7 +107,7 @@ public class StringToFile extends Abstra
if (fileParent == null || (fileParent.exists() && fileParent.isDirectory() && fileParent.canWrite())) {
FileUtils.writeStringToFile(file, content, charset, append);
} else {
- log.error("The parent file of {}Â doesn't exist or is not writable", file);
+ log.error("The parent file of {} doesn't exist or is not writable", file);
return false;
}
} finally {
@@ -126,14 +126,15 @@ public class StringToFile extends Abstra
boolean executionResult;
try {
executionResult = this.writeToFile();
- } catch (UnsupportedCharsetException ue) {
+ } catch (UnsupportedCharsetException ue) { // NOSONAR
executionResult = false;
- log.error("The encoding of file is not supported");
- } catch (IllegalCharsetNameException ie) {
+ log.error("The encoding of file is not supported", ue);
+ } catch (IllegalCharsetNameException ie) { // NOSONAR
executionResult = false;
- log.error("The encoding of file contains illegal characters");
- } catch (IOException e) {
+ log.error("The encoding of file contains illegal characters", ie);
+ } catch (IOException e) { // NOSONAR
executionResult = false;
+ log.error("IOException occurred", e);
}
return String.valueOf(executionResult);
}