This is the mail archive of the
java-patches@gcc.gnu.org
mailing list for the Java project.
[FYI/JDWP] shutting down in PacketProcessor
- From: Aaron Luchko <aluchko at redhat dot com>
- To: GCJ Patches <java-patches at gcc dot gnu dot org>
- Cc: classpath patches <classpath-patches at gnu dot org>
- Date: Tue, 12 Jul 2005 15:30:03 -0400
- Subject: [FYI/JDWP] shutting down in PacketProcessor
I've committed this patch to the trunk. This makes a minor improvement
in the shutdown sequence for jdwp by having the Jdwp object itself
handle the shutdown from the top level instead of PacketProcessor
telling the collection (and possibly the whole VM) to exit itself.
Aaron
ChangeLog
2005-07-12 Aaron Luchko <aluchko@redhat.com>
* gnu/classpath/jdwp/processor/PacketProcessor.java (run): Send
shutdown to jdwp instead of connection.
Index: gnu/classpath/jdwp/processor/PacketProcessor.java
===================================================================
RCS file: /cvsroot/classpath/classpath/gnu/classpath/jdwp/processor/PacketProcessor.java,v
retrieving revision 1.3
diff -u -p -r1.3 PacketProcessor.java
--- gnu/classpath/jdwp/processor/PacketProcessor.java 4 Jul 2005 16:03:23 -0000 1.3
+++ gnu/classpath/jdwp/processor/PacketProcessor.java 12 Jul 2005 18:11:08 -0000
@@ -40,6 +40,7 @@ exception statement from your version. *
package gnu.classpath.jdwp.processor;
+import gnu.classpath.jdwp.Jdwp;
import gnu.classpath.jdwp.JdwpConstants;
import gnu.classpath.jdwp.exception.JdwpException;
import gnu.classpath.jdwp.transport.JdwpCommandPacket;
@@ -146,8 +147,8 @@ public class PacketProcessor
{
ex.printStackTrace();
}
- // Time to shutdown, tell the _connection thread to stop reading
- _connection.shutdown();
+ // Time to shutdown, tell Jdwp to shutdown
+ Jdwp.getDefault().shutdown();
}
/**