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 2016/12/21 16:42:48 UTC
svn commit: r1775497 -
/jmeter/trunk/src/components/org/apache/jmeter/extractor/json/render/RenderAsJsonRenderer.java
Author: pmouawad
Date: Wed Dec 21 16:42:48 2016
New Revision: 1775497
URL: http://svn.apache.org/viewvc?rev=1775497&view=rev
Log:
Sonar : Fix code smells
Modified:
jmeter/trunk/src/components/org/apache/jmeter/extractor/json/render/RenderAsJsonRenderer.java
Modified: jmeter/trunk/src/components/org/apache/jmeter/extractor/json/render/RenderAsJsonRenderer.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/extractor/json/render/RenderAsJsonRenderer.java?rev=1775497&r1=1775496&r2=1775497&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/extractor/json/render/RenderAsJsonRenderer.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/extractor/json/render/RenderAsJsonRenderer.java Wed Dec 21 16:42:48 2016
@@ -120,7 +120,7 @@ public class RenderAsJsonRenderer implem
private String process(String textToParse) {
try {
List<Object> matchStrings = extractWithJSonPath(textToParse, jsonPathExpressionField.getText());
- if (matchStrings.size() == 0) {
+ if (matchStrings.isEmpty()) {
return "NO MATCH"; //$NON-NLS-1$
} else {
StringBuilder builder = new StringBuilder();
@@ -260,6 +260,7 @@ public class RenderAsJsonRenderer implem
/** {@inheritDoc} */
@Override
public void setBackgroundColor(Color backGround) {
+ // NOOP
}
}