Constify some commands in thread.c
[deliverable/binutils-gdb.git] / gdb / thread.c
index fce37c5cb8080b608ab089e35421047bcbf10179..91c077a4909ba40a595a985b2caf098dbf655a57 100644 (file)
 #include "thread-fsm.h"
 #include "tid-parse.h"
 #include <algorithm>
+#include "common/gdb_optional.h"
 
 /* Definition of struct thread_info exported to gdbthread.h.  */
 
-/* Prototypes for exported functions.  */
-
-void _initialize_thread (void);
-
 /* Prototypes for local functions.  */
 
 struct thread_info *thread_list = NULL;
@@ -522,6 +519,7 @@ find_thread_id (struct inferior *inf, int thr_num)
 }
 
 /* Find a thread_info by matching PTID.  */
+
 struct thread_info *
 find_thread_ptid (ptid_t ptid)
 {
@@ -534,6 +532,17 @@ find_thread_ptid (ptid_t ptid)
   return NULL;
 }
 
+/* See gdbthread.h.  */
+
+struct thread_info *
+find_thread_by_handle (struct value *thread_handle, struct inferior *inf)
+{
+  return target_thread_handle_to_thread_info
+          (value_contents_all (thread_handle),
+           TYPE_LENGTH (value_type (thread_handle)),
+           inf);
+}
+
 /*
  * Thread iterator function.
  *
@@ -1240,7 +1249,6 @@ print_thread_info_1 (struct ui_out *uiout, char *requested_threads,
 {
   struct thread_info *tp;
   ptid_t current_ptid;
-  struct cleanup *old_chain;
   const char *extra_info, *name, *target_id;
   struct inferior *inf;
   int default_inf_num = current_inferior ()->num;
@@ -1248,147 +1256,143 @@ print_thread_info_1 (struct ui_out *uiout, char *requested_threads,
   update_thread_list ();
   current_ptid = inferior_ptid;
 
-  /* We'll be switching threads temporarily.  */
-  old_chain = make_cleanup_restore_current_thread ();
-
-  /* For backward compatibility, we make a list for MI.  A table is
-     preferable for the CLI, though, because it shows table
-     headers.  */
-  if (uiout->is_mi_like_p ())
-    make_cleanup_ui_out_list_begin_end (uiout, "threads");
-  else
-    {
-      int n_threads = 0;
-
-      for (tp = thread_list; tp; tp = tp->next)
-       {
-         if (!should_print_thread (requested_threads, default_inf_num,
-                                   global_ids, pid, tp))
-           continue;
+  {
+    /* For backward compatibility, we make a list for MI.  A table is
+       preferable for the CLI, though, because it shows table
+       headers.  */
+    gdb::optional<ui_out_emit_list> list_emitter;
+    gdb::optional<ui_out_emit_table> table_emitter;
+
+    if (uiout->is_mi_like_p ())
+      list_emitter.emplace (uiout, "threads");
+    else
+      {
+       int n_threads = 0;
 
-         ++n_threads;
-       }
+       for (tp = thread_list; tp; tp = tp->next)
+         {
+           if (!should_print_thread (requested_threads, default_inf_num,
+                                     global_ids, pid, tp))
+             continue;
 
-      if (n_threads == 0)
-       {
-         if (requested_threads == NULL || *requested_threads == '\0')
-           uiout->message (_("No threads.\n"));
-         else
-           uiout->message (_("No threads match '%s'.\n"),
-                           requested_threads);
-         do_cleanups (old_chain);
-         return;
-       }
+           ++n_threads;
+         }
 
-      if (show_global_ids || uiout->is_mi_like_p ())
-       make_cleanup_ui_out_table_begin_end (uiout, 5, n_threads, "threads");
-      else
-       make_cleanup_ui_out_table_begin_end (uiout, 4, n_threads, "threads");
+       if (n_threads == 0)
+         {
+           if (requested_threads == NULL || *requested_threads == '\0')
+             uiout->message (_("No threads.\n"));
+           else
+             uiout->message (_("No threads match '%s'.\n"),
+                             requested_threads);
+           return;
+         }
 
-      uiout->table_header (1, ui_left, "current", "");
+       table_emitter.emplace (uiout, show_global_ids ? 5 : 4,
+                              n_threads, "threads");
 
-      if (!uiout->is_mi_like_p ())
+       uiout->table_header (1, ui_left, "current", "");
        uiout->table_header (4, ui_left, "id-in-tg", "Id");
-      if (show_global_ids || uiout->is_mi_like_p ())
-       uiout->table_header (4, ui_left, "id", "GId");
-      uiout->table_header (17, ui_left, "target-id", "Target Id");
-      uiout->table_header (1, ui_left, "frame", "Frame");
-      uiout->table_body ();
-    }
+       if (show_global_ids)
+         uiout->table_header (4, ui_left, "id", "GId");
+       uiout->table_header (17, ui_left, "target-id", "Target Id");
+       uiout->table_header (1, ui_left, "frame", "Frame");
+       uiout->table_body ();
+      }
 
-  ALL_THREADS_BY_INFERIOR (inf, tp)
-    {
-      int core;
+    /* We'll be switching threads temporarily.  */
+    scoped_restore_current_thread restore_thread;
 
-      if (!should_print_thread (requested_threads, default_inf_num,
-                               global_ids, pid, tp))
-       continue;
+    ALL_THREADS_BY_INFERIOR (inf, tp)
+      {
+       int core;
 
-      ui_out_emit_tuple tuple_emitter (uiout, NULL);
+       if (!should_print_thread (requested_threads, default_inf_num,
+                                 global_ids, pid, tp))
+         continue;
 
-      if (!uiout->is_mi_like_p ())
-       {
-         if (tp->ptid == current_ptid)
-           uiout->field_string ("current", "*");
-         else
-           uiout->field_skip ("current");
-       }
+       ui_out_emit_tuple tuple_emitter (uiout, NULL);
 
-      if (!uiout->is_mi_like_p ())
-       uiout->field_string ("id-in-tg", print_thread_id (tp));
+       if (!uiout->is_mi_like_p ())
+         {
+           if (tp->ptid == current_ptid)
+             uiout->field_string ("current", "*");
+           else
+             uiout->field_skip ("current");
 
-      if (show_global_ids || uiout->is_mi_like_p ())
-       uiout->field_int ("id", tp->global_num);
+           uiout->field_string ("id-in-tg", print_thread_id (tp));
+         }
 
-      /* 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 (show_global_ids || uiout->is_mi_like_p ())
+         uiout->field_int ("id", tp->global_num);
 
-      target_id = target_pid_to_str (tp->ptid);
-      extra_info = target_extra_thread_info (tp);
-      name = tp->name ? tp->name : target_thread_name (tp);
+       /* 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_id);
-         if (extra_info)
-           uiout->field_string ("details", extra_info);
-         if (name)
-           uiout->field_string ("name", name);
-       }
-      else
-       {
-         struct cleanup *str_cleanup;
-         char *contents;
-
-         if (extra_info && name)
-           contents = xstrprintf ("%s \"%s\" (%s)", target_id,
-                                  name, extra_info);
-         else if (extra_info)
-           contents = xstrprintf ("%s (%s)", target_id, extra_info);
-         else if (name)
-           contents = xstrprintf ("%s \"%s\"", target_id, name);
-         else
-           contents = xstrdup (target_id);
-         str_cleanup = make_cleanup (xfree, contents);
+       target_id = target_pid_to_str (tp->ptid);
+       extra_info = target_extra_thread_info (tp);
+       name = tp->name ? tp->name : target_thread_name (tp);
 
-         uiout->field_string ("target-id", contents);
-         do_cleanups (str_cleanup);
-       }
+       if (uiout->is_mi_like_p ())
+         {
+           uiout->field_string ("target-id", target_id);
+           if (extra_info)
+             uiout->field_string ("details", extra_info);
+           if (name)
+             uiout->field_string ("name", name);
+         }
+       else
+         {
+           std::string contents;
+
+           if (extra_info && name)
+             contents = string_printf ("%s \"%s\" (%s)", target_id,
+                                       name, extra_info);
+           else if (extra_info)
+             contents = string_printf ("%s (%s)", target_id, extra_info);
+           else if (name)
+             contents = string_printf ("%s \"%s\"", target_id, name);
+           else
+             contents = target_id;
+
+           uiout->field_string ("target-id", contents.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->ptid);
-         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->ptid);
+           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_int ("core", core);
-    }
+       core = target_core_of_thread (tp->ptid);
+       if (uiout->is_mi_like_p () && core != -1)
+         uiout->field_int ("core", core);
+      }
 
-  /* Restores the current thread and the frame selected before
-     the "info threads" command.  */
-  do_cleanups (old_chain);
+    /* This end scope restores the current thread and the frame
+       selected before the "info threads" command, and it finishes the
+       ui-out list or table.  */
+  }
 
   if (pid == -1 && requested_threads == NULL)
     {
@@ -1559,70 +1563,43 @@ restore_selected_frame (struct frame_id a_frame_id, int frame_level)
     }
 }
 
-/* Data used by the cleanup installed by
-   'make_cleanup_restore_current_thread'.  */
-
-struct current_thread_cleanup
-{
-  thread_info *thread;
-  struct frame_id selected_frame_id;
-  int selected_frame_level;
-  int was_stopped;
-  inferior *inf;
-};
-
-static void
-do_restore_current_thread_cleanup (void *arg)
+scoped_restore_current_thread::~scoped_restore_current_thread ()
 {
-  struct current_thread_cleanup *old = (struct current_thread_cleanup *) arg;
-
   /* If an entry of thread_info was previously selected, it won't be
      deleted because we've increased its refcount.  The thread represented
      by this thread_info entry may have already exited (due to normal exit,
      detach, etc), so the thread_info.state is THREAD_EXITED.  */
-  if (old->thread != NULL
+  if (m_thread != NULL
       /* If the previously selected thread belonged to a process that has
         in the mean time exited (or killed, detached, etc.), then don't revert
         back to it, but instead simply drop back to no thread selected.  */
-      && old->inf->pid != 0)
-    switch_to_thread (old->thread);
+      && m_inf->pid != 0)
+    switch_to_thread (m_thread);
   else
     {
       switch_to_no_thread ();
-      set_current_inferior (old->inf);
+      set_current_inferior (m_inf);
     }
 
   /* The running state of the originally selected thread may have
      changed, so we have to recheck it here.  */
   if (inferior_ptid != null_ptid
-      && old->was_stopped
+      && m_was_stopped
       && is_stopped (inferior_ptid)
       && target_has_registers
       && target_has_stack
       && target_has_memory)
-    restore_selected_frame (old->selected_frame_id,
-                           old->selected_frame_level);
-}
-
-static void
-restore_current_thread_cleanup_dtor (void *arg)
-{
-  struct current_thread_cleanup *old = (struct current_thread_cleanup *) arg;
-
-  if (old->thread != NULL)
-    old->thread->decref ();
+    restore_selected_frame (m_selected_frame_id, m_selected_frame_level);
 
-  old->inf->decref ();
-  xfree (old);
+  if (m_thread != NULL)
+    m_thread->decref ();
+  m_inf->decref ();
 }
 
-struct cleanup *
-make_cleanup_restore_current_thread (void)
+scoped_restore_current_thread::scoped_restore_current_thread ()
 {
-  struct current_thread_cleanup *old = XNEW (struct current_thread_cleanup);
-
-  old->thread = NULL;
-  old->inf = current_inferior ();
+  m_thread = NULL;
+  m_inf = current_inferior ();
 
   if (inferior_ptid != null_ptid)
     {
@@ -1631,8 +1608,8 @@ make_cleanup_restore_current_thread (void)
 
       gdb_assert (tp != NULL);
 
-      old->was_stopped = tp->state == THREAD_STOPPED;
-      if (old->was_stopped
+      m_was_stopped = tp->state == THREAD_STOPPED;
+      if (m_was_stopped
          && target_has_registers
          && target_has_stack
          && target_has_memory)
@@ -1647,17 +1624,14 @@ make_cleanup_restore_current_thread (void)
       else
        frame = NULL;
 
-      old->selected_frame_id = get_frame_id (frame);
-      old->selected_frame_level = frame_relative_level (frame);
+      m_selected_frame_id = get_frame_id (frame);
+      m_selected_frame_level = frame_relative_level (frame);
 
       tp->incref ();
-      old->thread = tp;
+      m_thread = tp;
     }
 
-  old->inf->incref ();
-
-  return make_cleanup_dtor (do_restore_current_thread_cleanup, old,
-                           restore_current_thread_cleanup_dtor);
+  m_inf->incref ();
 }
 
 /* See gdbthread.h.  */
@@ -1727,9 +1701,6 @@ tp_array_compar (const thread_info *a, const thread_info *b)
 static void
 thread_apply_all_command (char *cmd, int from_tty)
 {
-  struct cleanup *old_chain;
-  char *saved_cmd;
-
   tp_array_compar_ascending = false;
   if (cmd != NULL
       && check_for_argument (&cmd, "-ascending", strlen ("-ascending")))
@@ -1743,12 +1714,9 @@ thread_apply_all_command (char *cmd, int from_tty)
 
   update_thread_list ();
 
-  old_chain = make_cleanup_restore_current_thread ();
-
   /* Save a copy of the command in case it is clobbered by
      execute_command.  */
-  saved_cmd = xstrdup (cmd);
-  make_cleanup (xfree, saved_cmd);
+  std::string saved_cmd = cmd;
 
   int tc = live_threads_count ();
   if (tc != 0)
@@ -1778,6 +1746,8 @@ thread_apply_all_command (char *cmd, int from_tty)
 
       std::sort (thr_list_cpy.begin (), thr_list_cpy.end (), tp_array_compar);
 
+      scoped_restore_current_thread restore_thread;
+
       for (thread_info *thr : thr_list_cpy)
        if (thread_alive (thr))
          {
@@ -1788,11 +1758,9 @@ thread_apply_all_command (char *cmd, int from_tty)
            execute_command (cmd, from_tty);
 
            /* Restore exact command used previously.  */
-           strcpy (cmd, saved_cmd);
+           strcpy (cmd, saved_cmd.c_str ());
          }
     }
-
-  do_cleanups (old_chain);
 }
 
 /* Implementation of the "thread apply" command.  */
@@ -1801,8 +1769,6 @@ static void
 thread_apply_command (char *tidlist, int from_tty)
 {
   char *cmd = NULL;
-  struct cleanup *old_chain;
-  char *saved_cmd;
   tid_range_parser parser;
 
   if (tidlist == NULL || *tidlist == '\000')
@@ -1828,10 +1794,9 @@ thread_apply_command (char *tidlist, int from_tty)
 
   /* Save a copy of the command in case it is clobbered by
      execute_command.  */
-  saved_cmd = xstrdup (cmd);
-  old_chain = make_cleanup (xfree, saved_cmd);
+  std::string saved_cmd = cmd;
 
-  make_cleanup_restore_current_thread ();
+  scoped_restore_current_thread restore_thread;
 
   parser.init (tidlist, current_inferior ()->num);
   while (!parser.finished () && parser.cur_tok () < cmd)
@@ -1886,10 +1851,8 @@ thread_apply_command (char *tidlist, int from_tty)
       execute_command (cmd, from_tty);
 
       /* Restore exact command used previously.  */
-      strcpy (cmd, saved_cmd);
+      strcpy (cmd, saved_cmd.c_str ());
     }
-
-  do_cleanups (old_chain);
 }
 
 /* Switch to the specified thread.  Will dispatch off to thread_apply_command
@@ -1949,7 +1912,7 @@ thread_command (char *tidstr, int from_tty)
 /* Implementation of `thread name'.  */
 
 static void
-thread_name_command (char *arg, int from_tty)
+thread_name_command (const char *arg, int from_tty)
 {
   struct thread_info *info;
 
@@ -1966,7 +1929,7 @@ thread_name_command (char *arg, int from_tty)
 /* Find thread ids with a name, target pid, or extra info matching ARG.  */
 
 static void
-thread_find_command (char *arg, int from_tty)
+thread_find_command (const char *arg, int from_tty)
 {
   struct thread_info *tp;
   const char *tmp;
This page took 0.031594 seconds and 4 git commands to generate.