gdb: make inferior::clear_thread_list always silent
authorPedro Alves <pedro@palves.net>
Mon, 12 Dec 2022 20:30:59 +0000 (20:30 +0000)
committerAndrew Burgess <aburgess@redhat.com>
Wed, 23 Aug 2023 08:57:38 +0000 (09:57 +0100)
commit604fe2d97f7df4343cace1c2bc42f8726a0657bf
tree5131e8414b938216d8f73428fd65b3b607382cfc
parent951dbdfeec44d8d2971b155f17edff09d6f1a67a
gdb: make inferior::clear_thread_list always silent

After this commit:

  commit a78ef8757418105c35685c5d82b9fdf79459321b
  Date:   Wed Jun 22 18:10:00 2022 +0100

      Always emit =thread-exited notifications, even if silent

The function mi_interp::on_thread_exited (or mi_thread_exit as the
function was called back then) no longer makes use of the "silent"
parameter.

As a result there is no difference between inferior::clear_thread_list
with silent true or false, because:

  - None of the interpreter ::on_thread_exited functions rely on the
    silent parameter, and

  - None of GDB's thread_exit observers rely on the silent parameter
  either.

This commit removes the silent parameter from
inferior::clear_thread_list, and makes the function always silent.

This commit was originally part of a larger series:

  https://inbox.sourceware.org/gdb-patches/20221212203101.1034916-1-pedro@palves.net/

But didn't really need to be part of that series.  I had an interest
in seeing this patch merged:

  https://inbox.sourceware.org/gdb-patches/20221212203101.1034916-31-pedro@palves.net/

Which also didn't really need to be part of the larger series, but
does depend, at least a little, on this commit.  In order to get the
fix I'm interested in merged quicker, I (Andrew Burgess) have rebased
this commit outside of the original series.  Any bugs introduced while
splitting this patch out and rebasing, are entirely my own.

There should be no user visible changes after this commit.

Co-Authored-By: Andrew Burgess <aburgess@redhat.com>
gdb/inferior.c
gdb/inferior.h
gdb/thread.c