gdb: remove unneeded argument in check_multi_target_resumption
authorSimon Marchi <simon.marchi@polymtl.ca>
Wed, 13 Jan 2021 04:42:12 +0000 (23:42 -0500)
committerPedro Alves <pedro@palves.net>
Wed, 17 Mar 2021 13:05:55 +0000 (13:05 +0000)
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
gdb/infrun.c

index 7a824a8a0cc1eb6df33752d3882903656da5b018..a606f54009c70d9fe940830eaea699979b7395f3 100644 (file)
@@ -1,3 +1,8 @@
+2021-03-17  Simon Marchi  <simon.marchi@polymtl.ca>
+
+       * infrun.c (check_multi_target_resumption): Remove argument to
+       all_non_exited_inferiors.
+
 2021-03-16  Christian Biesinger  <cbiesinger@google.com>
 
        * windows-nat.c (windows_init_thread_list): Add message to
index a271220b2616dc8da9f1ee3645a01608afc2c477..3b65a6de9fe21159b32ba0455c0146ed8196ac99 100644 (file)
@@ -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);