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 c71c2fb..4d597a6 100644 (file)
--- a/libjava/ChangeLog
+++ b/libjava/ChangeLog
@@ -1,5 +1,10 @@
 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.
 
 1999-08-18  Kresten Krab Thorup  <krab@gnu.org>

diff --git a/libjava/java/lang/ThreadGroup.java b/libjava/java/lang/ThreadGroup.java
index 1aa1a9a..41a017d 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 9dc192e..9e663ba 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);
 }