From f058c5210f7189e3fdc7a3b16ff89ba4f00d339f Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Tue, 12 Jan 2021 23:42:12 -0500 Subject: [PATCH] gdb: remove unneeded argument in check_multi_target_resumption If we reach the modified line, resume_target is necessarily nullptr, because of the check at the beginning of the function. So we'll necessarily iterate on all non-exited inferiors (across all targets), which is what we want. So just remove the unnecessary argument. gdb/ChangeLog: * infrun.c (check_multi_target_resumption): Remove argument to all_non_exited_inferiors. Change-Id: If95704915dca19599d5f7f4732bbd6ccd20bf6b4 --- gdb/ChangeLog | 5 +++++ gdb/infrun.c | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 7a824a8a0cc..a606f54009c 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2021-03-17 Simon Marchi + + * infrun.c (check_multi_target_resumption): Remove argument to + all_non_exited_inferiors. + 2021-03-16 Christian Biesinger * windows-nat.c (windows_init_thread_list): Add message to diff --git a/gdb/infrun.c b/gdb/infrun.c index a271220b261..3b65a6de9fe 100644 --- a/gdb/infrun.c +++ b/gdb/infrun.c @@ -2809,7 +2809,7 @@ check_multi_target_resumption (process_stratum_target *resume_target) always-non-stop mode. */ inferior *first_not_non_stop = nullptr; - for (inferior *inf : all_non_exited_inferiors (resume_target)) + for (inferior *inf : all_non_exited_inferiors ()) { switch_to_inferior_no_thread (inf); -- 2.30.2