From d9916b766214af534a15b616df5d887514e91ed7 Mon Sep 17 00:00:00 2001 From: gaudryc Date: Sun, 6 Jan 2019 19:18:27 +0100 Subject: [PATCH] Bug: Useless control flow to next line in com.bwssystems.HABridge.plugins.http.HttpTestHandler.updateTheData(String, String) This method contains a useless control flow statement in which control flow follows to the same or following line regardless of whether or not the branch is taken. Often, this is caused by inadvertently using an empty statement as the body of an if statement, e.g.: --- .../com/bwssystems/HABridge/plugins/http/HttpTestHandler.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/com/bwssystems/HABridge/plugins/http/HttpTestHandler.java b/src/main/java/com/bwssystems/HABridge/plugins/http/HttpTestHandler.java index 10d7ceb..c54d3b0 100644 --- a/src/main/java/com/bwssystems/HABridge/plugins/http/HttpTestHandler.java +++ b/src/main/java/com/bwssystems/HABridge/plugins/http/HttpTestHandler.java @@ -32,7 +32,7 @@ public class HttpTestHandler extends HTTPHandler { } else { for(NameValue aTest:this.theData) { - if(aTest.getName().equals(compareValue)); + if(aTest.getName().equals(compareValue)) aTest.setValue(testData); } }