Hi,

This is the final cleanup for a while (startup synchronization aside). It simply removes the whole main thread notion from JDWP. Instead, the VM will handle this when sending VM_INIT. [Aka: I changed my mind!]

Keith

ChangeLog
2006-03-16  Keith Seitz  <[EMAIL PROTECTED]>

        * gnu/classpath/jdwp/Jdwp.java (_mainThread): Not needed. Removed
        all references.
        (run): Remove catch clause for InterruptedException. It is no
        longer necessary.
Index: Jdwp.java
===================================================================
RCS file: /sources/classpath/classpath/gnu/classpath/jdwp/Jdwp.java,v
retrieving revision 1.5
diff -u -r1.5 Jdwp.java
--- Jdwp.java	16 Mar 2006 21:06:08 -0000	1.5
+++ Jdwp.java	16 Mar 2006 23:03:01 -0000
@@ -80,9 +80,6 @@
   // (-Xrunjdwp:..suspend=<boolean>)
   private static final String _PROPERTY_SUSPEND = "suspend";
 
-  // User's main application thread
-  private Thread _mainThread;
-
   // Connection to debugger
   private JdwpConnection _connection;
 
@@ -141,11 +138,9 @@
    * Configures the back-end
    *
    * @param configArgs  a string of configury options
-   * @param mainThread  the main application thread
    */
-  public void configure (String configArgs, Thread mainThread)
+  public void configure (String configArgs)
   {
-    _mainThread = mainThread;
     _processConfigury (configArgs);
   }
 
@@ -281,17 +276,6 @@
     try
       {
 	_doInitialization ();
-
-	_mainThread.start ();
-
-	_mainThread.join ();
-      }
-    catch (InterruptedException ie)
-      {
-	/* Shutting down. If we're in server mode, we should
-	   prepare for a new connection. Otherwise, we should
-	   simply exit. */
-	// FIXME
       }
     catch (Throwable t)
       {

Reply via email to