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 2012/08/19 23:50:01 UTC
svn commit: r1374864 -
/jmeter/trunk/src/core/org/apache/jmeter/threads/ThreadGroup.java
Author: pmouawad
Date: Sun Aug 19 21:50:01 2012
New Revision: 1374864
URL: http://svn.apache.org/viewvc?rev=1374864&view=rev
Log:
&= is needless as stoppedAll is true
Added TODO waiting for answer from sebb on dev mailing list
Bugzilla Id: 53418
Modified:
jmeter/trunk/src/core/org/apache/jmeter/threads/ThreadGroup.java
Modified: jmeter/trunk/src/core/org/apache/jmeter/threads/ThreadGroup.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/threads/ThreadGroup.java?rev=1374864&r1=1374863&r2=1374864&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/threads/ThreadGroup.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/threads/ThreadGroup.java Sun Aug 19 21:50:01 2012
@@ -429,7 +429,7 @@ public class ThreadGroup extends Abstrac
public boolean verifyThreadsStopped() {
boolean stoppedAll = true;
if (delayedStartup){
- stoppedAll &= verifyThreadStopped(threadStarter);
+ stoppedAll = verifyThreadStopped(threadStarter);
}
for (Thread t : allThreads.values()) {
stoppedAll = stoppedAll && verifyThreadStopped(t);
@@ -497,6 +497,7 @@ public class ThreadGroup extends Abstrac
try {
Thread.sleep(ms);
} catch (InterruptedException e) {
+ // TODO Is this silent exception intended
}
}