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/10/16 15:54:03 UTC

svn commit: r585142 - /jakarta/jmeter/trunk/src/core/org/apache/jmeter/JMeter.java

Author: sebb
Date: Tue Oct 16 06:53:47 2007
New Revision: 585142

URL: http://svn.apache.org/viewvc?rev=585142&view=rev
Log:
Tidy up; remove unused method and useless wait

Modified:
    jakarta/jmeter/trunk/src/core/org/apache/jmeter/JMeter.java

Modified: jakarta/jmeter/trunk/src/core/org/apache/jmeter/JMeter.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/JMeter.java?rev=585142&r1=585141&r2=585142&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/core/org/apache/jmeter/JMeter.java (original)
+++ jakarta/jmeter/trunk/src/core/org/apache/jmeter/JMeter.java Tue Oct 16 06:53:47 2007
@@ -596,26 +596,17 @@
 	
 	}
 
-	//TODO - is this needed?
-	public void startServer() {
-		startServer(0);
-	}
-
-	//TODO - should this be public?
-	public void startServer(int port) {
+	private void startServer(int port) {
 		try {
 			new RemoteJMeterEngineImpl(port);
-			while (true) {
-				Thread.sleep(Long.MAX_VALUE);
-			}
 		} catch (Exception ex) {
 			log.error("Giving up, as server failed with:", ex);
+			System.err.println("Server failed to start: "+ex);
 			System.exit(1);// Give up
 		}
 	}
 
-    // TODO - should this be public?
-	public void startNonGui(CLOption testFile, CLOption logFile, CLOption remoteStart)
+	private void startNonGui(CLOption testFile, CLOption logFile, CLOption remoteStart)
 			throws IllegalUserActionException {
 		// add a system property so samplers can check to see if JMeter
 		// is running in NonGui mode



---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org