Don't check target is running in remote_target::mourn_inferior
[deliverable/binutils-gdb.git] / gdb / thread.c
index 4a7fe689dbb8b80d40136ace4f7fea842c63fd42..630899c8cb86301127fc214680379f9d1680dfe0 100644 (file)
@@ -1,6 +1,6 @@
 /* Multi-process/thread control for GDB, the GNU debugger.
 
-   Copyright (C) 1986-2019 Free Software Foundation, Inc.
+   Copyright (C) 1986-2020 Free Software Foundation, Inc.
 
    Contributed by Lynx Real-Time Systems, Inc.  Los Gatos, CA.
 
@@ -444,7 +444,7 @@ thread_step_over_chain_remove (struct thread_info *tp)
   step_over_chain_remove (&step_over_queue_head, tp);
 }
 
-/* Delete the thread referenced by THR.  If SILENT, don't notifyi
+/* Delete the thread referenced by THR.  If SILENT, don't notify
    the observer of this exit.
    
    THR must not be NULL or a failed assertion will be raised.  */
@@ -709,7 +709,7 @@ delete_exited_threads (void)
       delete_thread (tp);
 }
 
-/* Return true value if stack temporaies are enabled for the thread
+/* Return true value if stack temporaries are enabled for the thread
    TP.  */
 
 bool
@@ -1090,85 +1090,85 @@ print_thread_info_1 (struct ui_out *uiout, const char *requested_threads,
 
     for (inferior *inf : all_inferiors ())
       for (thread_info *tp : inf->threads ())
-      {
-       int core;
+       {
+         int core;
 
-       any_thread = true;
-       if (tp == current_thread && tp->state == THREAD_EXITED)
-         current_exited = true;
+         any_thread = true;
+         if (tp == current_thread && tp->state == THREAD_EXITED)
+           current_exited = true;
 
-       if (!should_print_thread (requested_threads, default_inf_num,
-                                 global_ids, pid, tp))
-         continue;
+         if (!should_print_thread (requested_threads, default_inf_num,
+                                   global_ids, pid, tp))
+           continue;
 
-       ui_out_emit_tuple tuple_emitter (uiout, NULL);
+         ui_out_emit_tuple tuple_emitter (uiout, NULL);
 
-       if (!uiout->is_mi_like_p ())
-         {
-           if (tp == current_thread)
-             uiout->field_string ("current", "*");
-           else
-             uiout->field_skip ("current");
+         if (!uiout->is_mi_like_p ())
+           {
+             if (tp == current_thread)
+               uiout->field_string ("current", "*");
+             else
+               uiout->field_skip ("current");
 
-           uiout->field_string ("id-in-tg", print_thread_id (tp));
-         }
+             uiout->field_string ("id-in-tg", print_thread_id (tp));
+           }
 
-       if (show_global_ids || uiout->is_mi_like_p ())
-         uiout->field_signed ("id", tp->global_num);
+         if (show_global_ids || uiout->is_mi_like_p ())
+           uiout->field_signed ("id", tp->global_num);
 
-       /* For the CLI, we stuff everything into the target-id field.
-          This is a gross hack to make the output come out looking
-          correct.  The underlying problem here is that ui-out has no
-          way to specify that a field's space allocation should be
-          shared by several fields.  For MI, we do the right thing
-          instead.  */
+         /* For the CLI, we stuff everything into the target-id field.
+            This is a gross hack to make the output come out looking
+            correct.  The underlying problem here is that ui-out has no
+            way to specify that a field's space allocation should be
+            shared by several fields.  For MI, we do the right thing
+            instead.  */
 
-       if (uiout->is_mi_like_p ())
-         {
-           uiout->field_string ("target-id", target_pid_to_str (tp->ptid));
+         if (uiout->is_mi_like_p ())
+           {
+             uiout->field_string ("target-id", target_pid_to_str (tp->ptid));
 
-           const char *extra_info = target_extra_thread_info (tp);
-           if (extra_info != nullptr)
-             uiout->field_string ("details", extra_info);
+             const char *extra_info = target_extra_thread_info (tp);
+             if (extra_info != nullptr)
+               uiout->field_string ("details", extra_info);
 
-           const char *name = (tp->name != nullptr
-                               ? tp->name
-                               : target_thread_name (tp));
-           if (name != NULL)
-             uiout->field_string ("name", name);
-         }
-       else
-         {
-           uiout->field_string ("target-id",
-                                thread_target_id_str (tp).c_str ());
-         }
+             const char *name = (tp->name != nullptr
+                                 ? tp->name
+                                 : target_thread_name (tp));
+             if (name != NULL)
+               uiout->field_string ("name", name);
+           }
+         else
+           {
+             uiout->field_string ("target-id",
+                                  thread_target_id_str (tp).c_str ());
+           }
 
-       if (tp->state == THREAD_RUNNING)
-         uiout->text ("(running)\n");
-       else
-         {
-           /* The switch below puts us at the top of the stack (leaf
-              frame).  */
-           switch_to_thread (tp);
-           print_stack_frame (get_selected_frame (NULL),
-                              /* For MI output, print frame level.  */
-                              uiout->is_mi_like_p (),
-                              LOCATION, 0);
-         }
+         if (tp->state == THREAD_RUNNING)
+           uiout->text ("(running)\n");
+         else
+           {
+             /* The switch below puts us at the top of the stack (leaf
+                frame).  */
+             switch_to_thread (tp);
+             print_stack_frame (get_selected_frame (NULL),
+                                /* For MI output, print frame level.  */
+                                uiout->is_mi_like_p (),
+                                LOCATION, 0);
+           }
 
-       if (uiout->is_mi_like_p ())
-         {
-           const char *state = "stopped";
+         if (uiout->is_mi_like_p ())
+           {
+             const char *state = "stopped";
 
-           if (tp->state == THREAD_RUNNING)
-             state = "running";
-           uiout->field_string ("state", state);
-         }
+             if (tp->state == THREAD_RUNNING)
+               state = "running";
+             uiout->field_string ("state", state);
+           }
 
-       core = target_core_of_thread (tp->ptid);
-       if (uiout->is_mi_like_p () && core != -1)
-         uiout->field_signed ("core", core);
-      }
+         core = target_core_of_thread (tp->ptid);
+         if (uiout->is_mi_like_p () && core != -1)
+           uiout->field_signed ("core", core);
+       }
 
     /* This end scope restores the current thread and the frame
        selected before the "info threads" command, and it finishes the
@@ -1204,7 +1204,7 @@ print_thread_info (struct ui_out *uiout, const char *requested_threads,
 struct info_threads_opts
 {
   /* For "-gid".  */
-  int show_global_ids = 0;
+  bool show_global_ids = false;
 };
 
 static const gdb::option::option_def info_threads_option_defs[] = {
@@ -1375,7 +1375,8 @@ restore_selected_frame (struct frame_id a_frame_id, int frame_level)
     }
 }
 
-scoped_restore_current_thread::~scoped_restore_current_thread ()
+void
+scoped_restore_current_thread::restore ()
 {
   /* If an entry of thread_info was previously selected, it won't be
      deleted because we've increased its refcount.  The thread represented
@@ -1402,6 +1403,22 @@ scoped_restore_current_thread::~scoped_restore_current_thread ()
       && target_has_stack
       && target_has_memory)
     restore_selected_frame (m_selected_frame_id, m_selected_frame_level);
+}
+
+scoped_restore_current_thread::~scoped_restore_current_thread ()
+{
+  if (!m_dont_restore)
+    {
+      try
+       {
+         restore ();
+       }
+      catch (const gdb_exception &ex)
+       {
+         /* We're in a dtor, there's really nothing else we can do
+            but swallow the exception.  */
+       }
+    }
 
   if (m_thread != NULL)
     m_thread->decref ();
@@ -1574,7 +1591,7 @@ static const gdb::option::option_def thr_qcs_flags_option_defs[] = {
    ASCENDING and FLAGS as context.  */
 
 static inline std::array<gdb::option::option_def_group, 2>
-make_thread_apply_all_options_def_group (int *ascending,
+make_thread_apply_all_options_def_group (bool *ascending,
                                         qcs_flags *flags)
 {
   return {{
@@ -1603,7 +1620,7 @@ make_thread_apply_options_def_group (qcs_flags *flags)
 static void
 thread_apply_all_command (const char *cmd, int from_tty)
 {
-  int ascending = false;
+  bool ascending = false;
   qcs_flags flags;
 
   auto group = make_thread_apply_all_options_def_group (&ascending,
@@ -1818,6 +1835,8 @@ thread_apply_command (const char *tidlist, int from_tty)
 static void
 taas_command (const char *cmd, int from_tty)
 {
+  if (cmd == NULL || *cmd == '\0')
+    error (_("Please specify a command to apply on all threads"));
   std::string expanded = std::string ("thread apply all -s ") + cmd;
   execute_command (expanded.c_str (), from_tty);
 }
@@ -1827,6 +1846,8 @@ taas_command (const char *cmd, int from_tty)
 static void
 tfaas_command (const char *cmd, int from_tty)
 {
+  if (cmd == NULL || *cmd == '\0')
+    error (_("Please specify a command to apply on all frames of all threads"));
   std::string expanded
     = std::string ("thread apply all -s -- frame apply all -s ") + cmd;
   execute_command (expanded.c_str (), from_tty);
@@ -1951,7 +1972,7 @@ thread_find_command (const char *arg, int from_tty)
 }
 
 /* Print notices when new threads are attached and detached.  */
-int print_thread_events = 1;
+bool print_thread_events = true;
 static void
 show_print_thread_events (struct ui_file *file, int from_tty,
                          struct cmd_list_element *c, const char *value)
This page took 0.039526 seconds and 4 git commands to generate.