ThreadGroup.java: Fixed now-erroneous comment.

	* java/lang/ThreadGroup.java: Fixed now-erroneous comment.
	* java/lang/natThread.cc (finish_): Call ThreadGroup.remove.

From-SVN: r28753

diff --git a/libjava/ChangeLog b/libjava/ChangeLog
index c71c2fb6b2439a0a8bd25b3848af324c682e578d..4d597a6e235322ad1d45a18c34f8a2b58553bff9 100644 (file)
--- a/libjava/ChangeLog
+++ b/libjava/ChangeLog
@@ -1,3 +1,8 @@
+1999-08-18  Tom Tromey  <tromey@cygnus.com>
+
+       * java/lang/ThreadGroup.java: Fixed now-erroneous comment.
+       * java/lang/natThread.cc (finish_): Call ThreadGroup.remove.
+
 1999-08-18  Tom Tromey  <tromey@cygnus.com>
 
        * include/javaprims.h ("Java"): Regenerated namespace decls.

diff --git a/libjava/java/lang/ThreadGroup.java b/libjava/java/lang/ThreadGroup.java
index 1aa1a9a3bedd7592cd5b683e3957a160a262e34c..41a017d8e3bd7a28e16596668ed14debb9d5efc9 100644 (file)
--- a/libjava/java/lang/ThreadGroup.java
+++ b/libjava/java/lang/ThreadGroup.java
@@ -80,9 +80,7 @@ public class ThreadGroup
   }
 
   // This is called by the Thread code to remove a Thread from our
-  // internal list.  FIXME: currently, it isn't called at all.  There
-  // doesn't appear to be any way to remove a Thread from a
-  // ThreadGroup (except the unimplemented destroy method).
+  // internal list.
   final void remove (Thread t)
   {
     threads.removeElement(t);

diff --git a/libjava/java/lang/natThread.cc b/libjava/java/lang/natThread.cc
index 9dc192e12c4ff3182d43100d0996a2a306ece9d6..9e663ba3ced01a61d3968ff016a8874c0bdae73a 100644 (file)
--- a/libjava/java/lang/natThread.cc
+++ b/libjava/java/lang/natThread.cc
@@ -250,6 +250,8 @@ java::lang::Thread::finish_ (void)
       _Jv_MutexUnlock (&t->interrupt_mutex);
     }
 
+  group->remove (this);
+
   _Jv_MonitorExit (this);
 }