You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by fs...@apache.org on 2017/01/28 12:17:20 UTC
svn commit: r1780668 -
/jmeter/trunk/src/components/org/apache/jmeter/control/RandomController.java
Author: fschumacher
Date: Sat Jan 28 12:17:20 2017
New Revision: 1780668
URL: http://svn.apache.org/viewvc?rev=1780668&view=rev
Log:
Use isEmpty instead of size on a list to check whether it is empty. Reorder the if/else clause to avoid negation.
Modified:
jmeter/trunk/src/components/org/apache/jmeter/control/RandomController.java
Modified: jmeter/trunk/src/components/org/apache/jmeter/control/RandomController.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/control/RandomController.java?rev=1780668&r1=1780667&r2=1780668&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/control/RandomController.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/control/RandomController.java Sat Jan 28 12:17:20 2017
@@ -35,10 +35,10 @@ public class RandomController extends In
*/
@Override
protected void resetCurrent() {
- if (getSubControllers().size() > 0) {
- current = ThreadLocalRandom.current().nextInt(this.getSubControllers().size());
- } else {
+ if (getSubControllers().isEmpty()) {
current = 0;
+ } else {
+ current = ThreadLocalRandom.current().nextInt(this.getSubControllers().size());
}
}