From 15f4dddd8a29d064a634140a5fcb830715681889 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Fri, 11 Dec 2020 09:21:53 -0700 Subject: [PATCH] Use thread_info_ref in enable_thread_stack_temporaries This changes enable_thread_stack_temporaries to use a thread_info_ref, removing some manual reference counting. gdb/ChangeLog 2020-12-11 Tom Tromey * gdbthread.h (class enable_thread_stack_temporaries) : Change type. --- gdb/ChangeLog | 5 +++++ gdb/gdbthread.h | 10 ++-------- 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 4533a6a7b8b..732a1cf0956 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2020-12-11 Tom Tromey + + * gdbthread.h (class enable_thread_stack_temporaries) : + Change type. + 2020-12-11 Tom Tromey * ada-tasks.c (struct ada_tasks_pspace_data) : New diff --git a/gdb/gdbthread.h b/gdb/gdbthread.h index e5484ac54ca..386b50a7e77 100644 --- a/gdb/gdbthread.h +++ b/gdb/gdbthread.h @@ -711,12 +711,8 @@ class enable_thread_stack_temporaries public: explicit enable_thread_stack_temporaries (struct thread_info *thr) - : m_thr (thr) + : m_thr (thread_info_ref::new_reference (thr)) { - gdb_assert (m_thr != NULL); - - m_thr->incref (); - m_thr->stack_temporaries_enabled = true; m_thr->stack_temporaries.clear (); } @@ -725,15 +721,13 @@ public: { m_thr->stack_temporaries_enabled = false; m_thr->stack_temporaries.clear (); - - m_thr->decref (); } DISABLE_COPY_AND_ASSIGN (enable_thread_stack_temporaries); private: - struct thread_info *m_thr; + thread_info_ref m_thr; }; extern bool thread_stack_temporaries_enabled_p (struct thread_info *tp); -- 2.30.2