From: Tom Tromey <tromey@cygnus.com>
Date: Wed, 18 Aug 1999 19:51:23 +0000 (+0000)
Subject: ThreadGroup.java: Fixed now-erroneous comment.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=5a0eb8326f73277b5a5aa2a6229899a31a998855;p=gcc.git

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 c71c2fb6b24..4d597a6e235 100644
--- 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 1aa1a9a3bed..41a017d8e3b 100644
--- 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 9dc192e12c4..9e663ba3ced 100644
--- 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);
 }