From 7c952b6daed25b7bb09e90f524fd4981b0f62284 Mon Sep 17 00:00:00 2001 From: Nicholas Duffek Date: Wed, 19 Jul 2000 21:03:06 +0000 Subject: * thread.c (free_thread): New function. (init_thread_list): Always zero highest_thread_num. Call free_thread() instead of free(). (delete_thread): Move thread cleanup code to free_thread(). --- gdb/ChangeLog | 7 +++++++ gdb/thread.c | 34 +++++++++++++++++++--------------- 2 files changed, 26 insertions(+), 15 deletions(-) (limited to 'gdb') diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 9624c4a..c2cd85f 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +2000-07-19 Nicholas Duffek + + * thread.c (free_thread): New function. + (init_thread_list): Always zero highest_thread_num. Call + free_thread() instead of free(). + (delete_thread): Move thread cleanup code to free_thread(). + 2000-07-19 Elena Zannoni Multiarch the sh target. diff --git a/gdb/thread.c b/gdb/thread.c index 38e74d4..00c6467 100644 --- a/gdb/thread.c +++ b/gdb/thread.c @@ -63,22 +63,38 @@ static void restore_current_thread (int); static void switch_to_thread (int pid); static void prune_threads (void); +static void +free_thread (struct thread_info *tp) +{ + /* NOTE: this will take care of any left-over step_resume breakpoints, + but not any user-specified thread-specific breakpoints. */ + if (tp->step_resume_breakpoint) + delete_breakpoint (tp->step_resume_breakpoint); + + /* FIXME: do I ever need to call the back-end to give it a + chance at this private data before deleting the thread? */ + if (tp->private) + free (tp->private); + + free (tp); +} + void init_thread_list () { struct thread_info *tp, *tpnext; + highest_thread_num = 0; if (!thread_list) return; for (tp = thread_list; tp; tp = tpnext) { tpnext = tp->next; - free (tp); + free_thread (tp); } thread_list = NULL; - highest_thread_num = 0; } /* add_thread now returns a pointer to the new thread_info, @@ -134,19 +150,7 @@ delete_thread (pid) else thread_list = tp->next; - /* NOTE: this will take care of any left-over step_resume breakpoints, - but not any user-specified thread-specific breakpoints. */ - if (tp->step_resume_breakpoint) - delete_breakpoint (tp->step_resume_breakpoint); - - /* FIXME: do I ever need to call the back-end to give it a - chance at this private data before deleting the thread? */ - if (tp->private) - free (tp->private); - - free (tp); - - return; + free_thread (tp); } static struct thread_info * -- cgit v1.1