Author: fschumacher
Date: Sat Jan 28 12:22:02 2017
New Revision: 1780670

URL: http://svn.apache.org/viewvc?rev=1780670&view=rev
Log:
Return result of condition directly instead of wrapping it in an if/else clause.

Modified:
    
jmeter/trunk/src/components/org/apache/jmeter/control/ThroughputController.java

Modified: 
jmeter/trunk/src/components/org/apache/jmeter/control/ThroughputController.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/control/ThroughputController.java?rev=1780670&r1=1780669&r2=1780670&view=diff
==============================================================================
--- 
jmeter/trunk/src/components/org/apache/jmeter/control/ThroughputController.java 
(original)
+++ 
jmeter/trunk/src/components/org/apache/jmeter/control/ThroughputController.java 
Sat Jan 28 12:22:02 2017
@@ -211,11 +211,9 @@ public class ThroughputController extend
     public boolean isDone() {
         if (subControllersAndSamplers.isEmpty()) {
             return true;
-        } else if (getStyle() == BYNUMBER && getExecutions() >= 
getMaxThroughputAsInt()
-                && current >= getSubControllers().size()) {
-            return true;
         } else {
-            return false;
+            return getStyle() == BYNUMBER && getExecutions() >= 
getMaxThroughputAsInt()
+                && current >= getSubControllers().size();
         }
     }
 


Reply via email to