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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
