From 5a0eb8326f73277b5a5aa2a6229899a31a998855 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Wed, 18 Aug 1999 19:51:23 +0000 Subject: [PATCH] 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 --- libjava/ChangeLog | 5 +++++ libjava/java/lang/ThreadGroup.java | 4 +--- libjava/java/lang/natThread.cc | 2 ++ 3 files changed, 8 insertions(+), 3 deletions(-) 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 + + * java/lang/ThreadGroup.java: Fixed now-erroneous comment. + * java/lang/natThread.cc (finish_): Call ThreadGroup.remove. + 1999-08-18 Tom Tromey * 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); } -- 2.30.2