You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jmeter-dev@jakarta.apache.org by se...@apache.org on 2007/08/20 15:41:34 UTC
svn commit: r567702 -
/jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/engine/ClientJMeterEngine.java
Author: sebb
Date: Mon Aug 20 06:41:33 2007
New Revision: 567702
URL: http://svn.apache.org/viewvc?rev=567702&view=rev
Log:
Remove unnecessary try/catch
Modified:
jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/engine/ClientJMeterEngine.java
Modified: jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/engine/ClientJMeterEngine.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/engine/ClientJMeterEngine.java?rev=567702&r1=567701&r2=567702&view=diff
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/engine/ClientJMeterEngine.java (original)
+++ jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/engine/ClientJMeterEngine.java Mon Aug 20 06:41:33 2007
@@ -31,8 +31,7 @@
import org.apache.log.Logger;
/**
- * @author unattributed
- * @version $Revision$ Updated on: $Date$
+ * Class to run remote tests from the client JMeter and collect remote samples
*/
public class ClientJMeterEngine implements JMeterEngine, Runnable {
private static final Logger log = LoggingManager.getLoggerForClass();
@@ -110,17 +109,9 @@
testListeners = new SearchByClass(TestListener.class);
sampleListeners = new ConvertListeners();
HashTree testTree = getTestTree();
- // TODO see bug 39792; should not do any harm to synch the code here
- // @see http://issues.apache.org/bugzilla/show_bug.cgi?id=39792
synchronized(testTree) {
testTree.traverse(testListeners);
- // TODO this is a temporary fix - see bug 23487
- // @see http://issues.apache.org/bugzilla/show_bug.cgi?id=23487
- try { // probably no longer needed, now that the code is synchronised
- testTree.traverse(sampleListeners);
- } catch (IndexOutOfBoundsException e) {
- log.warn("Error replacing sample listeners", e);
- }
+ testTree.traverse(sampleListeners);
}
try {
---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org