ubsan: bpf: left shift cannot be represented in type 'DI' (aka 'long')
[deliverable/binutils-gdb.git] / gdb / tui / tui-regs.c
index 451bcdfb364eedd59ea717fc567e0c9262126576..1d936f712b6e95406cfca45e60d0f93b0ddd6676 100644 (file)
@@ -32,7 +32,6 @@
 #include "target.h"
 #include "tui/tui-layout.h"
 #include "tui/tui-win.h"
-#include "tui/tui-windata.h"
 #include "tui/tui-wingeneral.h"
 #include "tui/tui-file.h"
 #include "tui/tui-regs.h"
 
 #include "gdb_curses.h"
 
+/* Get the register from the frame and return a printable
+   representation of it.  */
 
-/*****************************************
-** STATIC LOCAL FUNCTIONS FORWARD DECLS    **
-******************************************/
-static void
-tui_display_register (struct tui_data_element *data,
-                      struct tui_gen_win_info *win_info);
+static gdb::unique_xmalloc_ptr<char>
+tui_register_format (struct frame_info *frame, int regnum)
+{
+  struct gdbarch *gdbarch = get_frame_arch (frame);
 
-static enum tui_status tui_show_register_group (struct reggroup *group,
-                                               struct frame_info *frame,
-                                               int refresh_values_only);
+  string_file stream;
 
-static enum tui_status tui_get_register (struct frame_info *frame,
-                                        struct tui_data_element *data,
-                                        int regnum, int *changedp);
+  scoped_restore save_pagination
+    = make_scoped_restore (&pagination_enabled, 0);
+  scoped_restore save_stdout
+    = make_scoped_restore (&gdb_stdout, &stream);
 
+  gdbarch_print_registers_info (gdbarch, &stream, frame, regnum, 1);
 
+  /* Remove the possible \n.  */
+  std::string &str = stream.string ();
+  if (!str.empty () && str.back () == '\n')
+    str.resize (str.size () - 1);
 
-/*****************************************
-** PUBLIC FUNCTIONS                     **
-******************************************/
+  /* Expand tabs into spaces, since ncurses on MS-Windows doesn't.  */
+  return tui_expand_tabs (str.c_str ());
+}
 
-/* Answer the number of the last line in the regs display.  If there
-   are no registers (-1) is returned.  */
-int
-tui_last_regs_line_no (void)
+/* Get the register value from the given frame and format it for the
+   display.  When changep is set, check if the new register value has
+   changed with respect to the previous call.  */
+static void
+tui_get_register (struct frame_info *frame,
+                  struct tui_data_item_window *data, 
+                 int regnum, bool *changedp)
 {
-  int num_lines = (-1);
-
-  if (TUI_DATA_WIN->regs_content_count > 0)
+  if (changedp)
+    *changedp = false;
+  if (target_has_registers)
     {
-      num_lines = (TUI_DATA_WIN->regs_content_count
-                  / TUI_DATA_WIN->regs_column_count);
-      if (TUI_DATA_WIN->regs_content_count % TUI_DATA_WIN->regs_column_count)
-       num_lines++;
+      gdb::unique_xmalloc_ptr<char> new_content
+       = tui_register_format (frame, regnum);
+
+      if (changedp != NULL
+         && strcmp (data->content.get (), new_content.get ()) != 0)
+       *changedp = true;
+
+      data->content = std::move (new_content);
     }
+}
+
+/* See tui-regs.h.  */
+
+int
+tui_data_window::last_regs_line_no () const
+{
+  int num_lines = m_regs_content.size () / m_regs_column_count;
+  if (m_regs_content.size () % m_regs_column_count)
+    num_lines++;
   return num_lines;
 }
 
+/* See tui-regs.h.  */
 
-/* Answer the line number that the register element at element_no is
-   on.  If element_no is greater than the number of register elements
-   there are, -1 is returned.  */
 int
-tui_line_from_reg_element_no (int element_no)
+tui_data_window::line_from_reg_element_no (int element_no) const
 {
-  if (element_no < TUI_DATA_WIN->regs_content_count)
+  if (element_no < m_regs_content.size ())
     {
       int i, line = (-1);
 
       i = 1;
       while (line == (-1))
        {
-         if (element_no < TUI_DATA_WIN->regs_column_count * i)
+         if (element_no < m_regs_column_count * i)
            line = i - 1;
          else
            i++;
@@ -108,95 +126,60 @@ tui_line_from_reg_element_no (int element_no)
     return (-1);
 }
 
+/* See tui-regs.h.  */
 
-/* Answer the index of the first element in line_no.  If line_no is
-   past the register area (-1) is returned.  */
 int
-tui_first_reg_element_no_inline (int line_no)
+tui_data_window::first_reg_element_no_inline (int line_no) const
 {
-  if ((line_no * TUI_DATA_WIN->regs_column_count)
-      <= TUI_DATA_WIN->regs_content_count)
-    return (((line_no + 1) * TUI_DATA_WIN->regs_column_count) 
-           - TUI_DATA_WIN->regs_column_count);
+  if (line_no * m_regs_column_count <= m_regs_content.size ())
+    return ((line_no + 1) * m_regs_column_count) - m_regs_column_count;
   else
     return (-1);
 }
 
-
 /* Show the registers of the given group in the data window
    and refresh the window.  */
 void
-tui_show_registers (struct reggroup *group)
+tui_data_window::show_registers (struct reggroup *group)
 {
-  enum tui_status ret = TUI_FAILURE;
-
-  /* Make sure the curses mode is enabled.  */
-  tui_enable ();
-
-  /* Make sure the register window is visible.  If not, select an
-     appropriate layout.  */
-  if (TUI_DATA_WIN == NULL || !TUI_DATA_WIN->generic.is_visible)
-    tui_set_layout_by_name (DATA_NAME);
-
   if (group == 0)
     group = general_reggroup;
 
-  /* Say that registers should be displayed, even if there is a
-     problem.  */
-  TUI_DATA_WIN->display_regs = TRUE;
-
   if (target_has_registers && target_has_stack && target_has_memory)
     {
-      ret = tui_show_register_group (group, get_selected_frame (NULL),
-                                     group == TUI_DATA_WIN->current_group);
-    }
-  if (ret == TUI_FAILURE)
-    {
-      TUI_DATA_WIN->current_group = 0;
-      tui_erase_data_content (NO_REGS_STRING);
+      show_register_group (group, get_selected_frame (NULL),
+                          group == m_current_group);
+
+      /* Clear all notation of changed values.  */
+      for (auto &&data_item_win : m_regs_content)
+       data_item_win.highlight = false;
+      m_current_group = group;
     }
   else
     {
-      int i;
-
-      /* Clear all notation of changed values.  */
-      for (i = 0; i < TUI_DATA_WIN->regs_content_count; i++)
-       {
-         struct tui_gen_win_info *data_item_win;
-          struct tui_win_element *win;
-
-         data_item_win = &TUI_DATA_WIN->regs_content[i]
-            ->which_element.data_window;
-          win = data_item_win->content[0];
-          win->which_element.data.highlight = FALSE;
-       }
-      TUI_DATA_WIN->current_group = group;
-      tui_display_all_data ();
+      m_current_group = 0;
+      m_regs_content.clear ();
     }
+
+  rerender ();
 }
 
 
 /* Set the data window to display the registers of the register group
    using the given frame.  Values are refreshed only when
-   refresh_values_only is TRUE.  */
+   refresh_values_only is true.  */
 
-static enum tui_status
-tui_show_register_group (struct reggroup *group,
-                         struct frame_info *frame, 
-                        int refresh_values_only)
+void
+tui_data_window::show_register_group (struct reggroup *group,
+                                     struct frame_info *frame, 
+                                     bool refresh_values_only)
 {
   struct gdbarch *gdbarch = get_frame_arch (frame);
-  enum tui_status ret = TUI_FAILURE;
   int nr_regs;
-  int allocated_here = FALSE;
   int regnum, pos;
-  char title[80];
 
   /* Make a new title showing which group we display.  */
-  snprintf (title, sizeof (title) - 1, "Register group: %s",
-            reggroup_name (group));
-  xfree (TUI_DATA_WIN->generic.title);
-  TUI_DATA_WIN->generic.title = xstrdup (title);
+  title = string_printf ("Register group: %s", reggroup_name (group));
 
   /* See how many registers must be displayed.  */
   nr_regs = 0;
@@ -217,324 +200,310 @@ tui_show_register_group (struct reggroup *group,
       nr_regs++;
     }
 
-  if (TUI_DATA_WIN->regs_content_count > 0 && !refresh_values_only)
+  m_regs_content.resize (nr_regs);
+
+  /* Now set the register names and values.  */
+  pos = 0;
+  for (regnum = 0; regnum < gdbarch_num_cooked_regs (gdbarch); regnum++)
     {
-      tui_free_data_content (TUI_DATA_WIN->regs_content,
-                             TUI_DATA_WIN->regs_content_count);
-      TUI_DATA_WIN->regs_content_count = 0;
+      struct tui_data_item_window *data_item_win;
+      const char *name;
+
+      /* Must be in the group.  */
+      if (!gdbarch_register_reggroup_p (gdbarch, regnum, group))
+       continue;
+
+      /* If the register name is empty, it is undefined for this
+        processor, so don't display anything.  */
+      name = gdbarch_register_name (gdbarch, regnum);
+      if (name == 0 || *name == '\0')
+       continue;
+
+      data_item_win = &m_regs_content[pos];
+      if (data_item_win)
+       {
+         if (!refresh_values_only)
+           {
+             data_item_win->item_no = regnum;
+             data_item_win->name = name;
+             data_item_win->highlight = false;
+           }
+         tui_get_register (frame, data_item_win, regnum, 0);
+       }
+      pos++;
     }
+}
+
+/* See tui-regs.h.  */
+
+void
+tui_data_window::display_registers_from (int start_element_no)
+{
+  int j, item_win_width, cur_y;
 
-  if (TUI_DATA_WIN->regs_content_count <= 0)
+  int max_len = 0;
+  for (auto &&data_item_win : m_regs_content)
     {
-      TUI_DATA_WIN->regs_content = tui_alloc_content (nr_regs, DATA_WIN);
-      allocated_here = TRUE;
-      refresh_values_only = FALSE;
-    }
+      const char *p;
+      int len;
+
+      len = 0;
+      p = data_item_win.content.get ();
+      if (p != 0)
+       len = strlen (p);
 
-  if (TUI_DATA_WIN->regs_content != NULL)
+      if (len > max_len)
+       max_len = len;
+    }
+  item_win_width = max_len + 1;
+  int i = start_element_no;
+
+  m_regs_column_count = (width - 2) / item_win_width;
+  if (m_regs_column_count == 0)
+    m_regs_column_count = 1;
+  item_win_width = (width - 2) / m_regs_column_count;
+
+  /* Now create each data "sub" window, and write the display into
+     it.  */
+  cur_y = 1;
+  while (i < m_regs_content.size ()
+        && cur_y <= viewport_height)
     {
-      if (!refresh_values_only || allocated_here)
+      for (j = 0;
+          j < m_regs_column_count && i < m_regs_content.size ();
+          j++)
        {
-         TUI_DATA_WIN->generic.content = NULL;
-         TUI_DATA_WIN->generic.content_size = 0;
-         tui_add_content_elements (&TUI_DATA_WIN->generic, nr_regs);
-         TUI_DATA_WIN->regs_content = TUI_DATA_WIN->generic.content;
-         TUI_DATA_WIN->regs_content_count = nr_regs;
+         /* Create the window if necessary.  */
+         m_regs_content[i].resize (1, item_win_width,
+                                   (item_win_width * j) + 1, cur_y);
+         i++;          /* Next register.  */
        }
+      cur_y++;         /* Next row.  */
+    }
+}
 
-      /* Now set the register names and values.  */
-      pos = 0;
-      for (regnum = 0; regnum < gdbarch_num_cooked_regs (gdbarch); regnum++)
-        {
-         struct tui_gen_win_info *data_item_win;
-          struct tui_data_element *data;
-          const char *name;
-
-          /* Must be in the group.  */
-          if (!gdbarch_register_reggroup_p (gdbarch, regnum, group))
-            continue;
-
-         /* If the register name is empty, it is undefined for this
-            processor, so don't display anything.  */
-         name = gdbarch_register_name (gdbarch, regnum);
-         if (name == 0 || *name == '\0')
-           continue;
-
-         data_item_win =
-            &TUI_DATA_WIN->regs_content[pos]->which_element.data_window;
-          data = &data_item_win->content[0]->which_element.data;
-          if (data)
-            {
-              if (!refresh_values_only)
-                {
-                  data->item_no = regnum;
-                  data->name = name;
-                  data->highlight = FALSE;
-                }
-              tui_get_register (frame, data, regnum, 0);
-            }
-          pos++;
-       }
+/* See tui-regs.h.  */
 
-      TUI_DATA_WIN->generic.content_size =
-       TUI_DATA_WIN->regs_content_count + TUI_DATA_WIN->data_content_count;
-      ret = TUI_SUCCESS;
-    }
+void
+tui_data_window::display_reg_element_at_line (int start_element_no,
+                                             int start_line_no)
+{
+  int element_no = start_element_no;
 
-  return ret;
+  if (start_element_no != 0 && start_line_no != 0)
+    {
+      int last_line_no, first_line_on_last_page;
+
+      last_line_no = last_regs_line_no ();
+      first_line_on_last_page = last_line_no - (height - 2);
+      if (first_line_on_last_page < 0)
+       first_line_on_last_page = 0;
+
+      /* If the element_no causes us to scroll past the end of the
+        registers, adjust what element to really start the
+        display at.  */
+      if (start_line_no > first_line_on_last_page)
+       element_no = first_reg_element_no_inline (first_line_on_last_page);
+    }
+  display_registers_from (element_no);
 }
 
-/* Function to display the registers in the content from
-   'start_element_no' until the end of the register content or the end
-   of the display height.  No checking for displaying past the end of
-   the registers is done here.  */
-void
-tui_display_registers_from (int start_element_no)
+/* See tui-regs.h.  */
+
+int
+tui_data_window::display_registers_from_line (int line_no)
 {
-  if (TUI_DATA_WIN->regs_content != NULL
-      && TUI_DATA_WIN->regs_content_count > 0)
+  int element_no;
+
+  if (line_no < 0)
+    line_no = 0;
+  else
     {
-      int i = start_element_no;
-      int j, item_win_width, cur_y;
-
-      int max_len = 0;
-      for (i = 0; i < TUI_DATA_WIN->regs_content_count; i++)
-        {
-          struct tui_data_element *data;
-          struct tui_gen_win_info *data_item_win;
-          char *p;
-          int len;
-
-          data_item_win
-           = &TUI_DATA_WIN->regs_content[i]->which_element.data_window;
-          data = &data_item_win->content[0]->which_element.data;
-          len = 0;
-          p = data->content;
-          if (p != 0)
-            while (*p)
-              {
-                if (*p++ == '\t')
-                  len = 8 * ((len / 8) + 1);
-                else
-                  len++;
-              }
-
-          if (len > max_len)
-            max_len = len;
-        }
-      item_win_width = max_len + 1;
-      i = start_element_no;
-
-      TUI_DATA_WIN->regs_column_count =
-        (TUI_DATA_WIN->generic.width - 2) / item_win_width;
-      if (TUI_DATA_WIN->regs_column_count == 0)
-        TUI_DATA_WIN->regs_column_count = 1;
-      item_win_width =
-        (TUI_DATA_WIN->generic.width - 2) / TUI_DATA_WIN->regs_column_count;
-
-      /* Now create each data "sub" window, and write the display into
-        it.  */
-      cur_y = 1;
-      while (i < TUI_DATA_WIN->regs_content_count 
-            && cur_y <= TUI_DATA_WIN->generic.viewport_height)
+      /* Make sure that we don't display off the end of the
+        registers.  */
+      if (line_no >= last_regs_line_no ())
        {
-         for (j = 0;
-              j < TUI_DATA_WIN->regs_column_count
-                && i < TUI_DATA_WIN->regs_content_count;
-              j++)
-           {
-             struct tui_gen_win_info *data_item_win;
-             struct tui_data_element *data_element_ptr;
-
-             /* Create the window if necessary.  */
-             data_item_win = &TUI_DATA_WIN->regs_content[i]
-                ->which_element.data_window;
-             data_element_ptr = &data_item_win->content[0]->which_element.data;
-              if (data_item_win->handle != NULL
-                  && (data_item_win->height != 1
-                      || data_item_win->width != item_win_width
-                      || data_item_win->origin.x != (item_win_width * j) + 1
-                      || data_item_win->origin.y != cur_y))
-                {
-                  tui_delete_win (data_item_win->handle);
-                  data_item_win->handle = 0;
-                }
-                  
-             if (data_item_win->handle == NULL)
-               {
-                 data_item_win->height = 1;
-                 data_item_win->width = item_win_width;
-                 data_item_win->origin.x = (item_win_width * j) + 1;
-                 data_item_win->origin.y = cur_y;
-                 tui_make_window (data_item_win, DONT_BOX_WINDOW);
-                  scrollok (data_item_win->handle, FALSE);
-               }
-              touchwin (data_item_win->handle);
-
-             /* Get the printable representation of the register
-                 and display it.  */
-              tui_display_register (data_element_ptr, data_item_win);
-             i++;              /* Next register.  */
-           }
-         cur_y++;              /* Next row.  */
+         line_no = line_from_reg_element_no (m_regs_content.size () - 1);
+         if (line_no < 0)
+           line_no = 0;
        }
     }
+
+  element_no = first_reg_element_no_inline (line_no);
+  if (element_no < m_regs_content.size ())
+    display_reg_element_at_line (element_no, line_no);
+  else
+    line_no = (-1);
+
+  return line_no;
 }
 
 
-/* Function to display the registers in the content from
-   'start_element_no' on 'start_line_no' until the end of the register
-   content or the end of the display height.  This function checks
-   that we won't display off the end of the register display.  */
-static void
-tui_display_reg_element_at_line (int start_element_no,
-                                int start_line_no)
+/* Answer the index first element displayed.  If none are displayed,
+   then return (-1).  */
+int
+tui_data_window::first_data_item_displayed ()
 {
-  if (TUI_DATA_WIN->regs_content != NULL
-      && TUI_DATA_WIN->regs_content_count > 0)
+  for (int i = 0; i < m_regs_content.size (); i++)
     {
-      int element_no = start_element_no;
+      struct tui_gen_win_info *data_item_win;
 
-      if (start_element_no != 0 && start_line_no != 0)
-       {
-         int last_line_no, first_line_on_last_page;
-
-         last_line_no = tui_last_regs_line_no ();
-         first_line_on_last_page
-           = last_line_no - (TUI_DATA_WIN->generic.height - 2);
-         if (first_line_on_last_page < 0)
-           first_line_on_last_page = 0;
-
-         /* If there is no other data displayed except registers, and
-            the element_no causes us to scroll past the end of the
-            registers, adjust what element to really start the
-            display at.  */
-         if (TUI_DATA_WIN->data_content_count <= 0
-             && start_line_no > first_line_on_last_page)
-           element_no
-             = tui_first_reg_element_no_inline (first_line_on_last_page);
-       }
-      tui_display_registers_from (element_no);
+      data_item_win = &m_regs_content[i];
+      if (data_item_win->is_visible ())
+       return i;
     }
+
+  return -1;
 }
 
+/* See tui-regs.h.  */
 
+void
+tui_data_window::delete_data_content_windows ()
+{
+  for (auto &&win : m_regs_content)
+    win.handle.reset (nullptr);
+}
 
-/* Function to display the registers starting at line line_no in the
-   data window.  Answers the line number that the display actually
-   started from.  If nothing is displayed (-1) is returned.  */
-int
-tui_display_registers_from_line (int line_no, 
-                                int force_display)
+
+void
+tui_data_window::erase_data_content (const char *prompt)
 {
-  if (TUI_DATA_WIN->regs_content_count > 0)
+  werase (handle.get ());
+  check_and_display_highlight_if_needed ();
+  if (prompt != NULL)
     {
-      int line, element_no;
-
-      if (line_no < 0)
-       line = 0;
-      else if (force_display)
-       { /* If we must display regs (force_display is true), then
-            make sure that we don't display off the end of the
-            registers.  */
-         if (line_no >= tui_last_regs_line_no ())
-           {
-             if ((line = tui_line_from_reg_element_no (
-                TUI_DATA_WIN->regs_content_count - 1)) < 0)
-               line = 0;
-           }
-         else
-           line = line_no;
-       }
-      else
-       line = line_no;
+      int half_width = (width - 2) / 2;
+      int x_pos;
 
-      element_no = tui_first_reg_element_no_inline (line);
-      if (element_no
-         < TUI_DATA_WIN->regs_content_count)
-       tui_display_reg_element_at_line (element_no, line);
+      if (strlen (prompt) >= half_width)
+       x_pos = 1;
       else
-       line = (-1);
+       x_pos = half_width - strlen (prompt);
+      mvwaddstr (handle.get (), (height / 2), x_pos, (char *) prompt);
+    }
+  wrefresh (handle.get ());
+}
 
-      return line;
+/* See tui-regs.h.  */
+
+void
+tui_data_window::rerender ()
+{
+  if (m_regs_content.empty ())
+    erase_data_content (_("[ Register Values Unavailable ]"));
+  else
+    {
+      erase_data_content (NULL);
+      delete_data_content_windows ();
+      display_registers_from (0);
     }
+}
 
-  return (-1);                 /* Nothing was displayed.  */
+
+/* Scroll the data window vertically forward or backward.  */
+void
+tui_data_window::do_scroll_vertical (int num_to_scroll)
+{
+  int first_element_no;
+  int first_line = (-1);
+
+  first_element_no = first_data_item_displayed ();
+  if (first_element_no < m_regs_content.size ())
+    first_line = line_from_reg_element_no (first_element_no);
+  else
+    { /* Calculate the first line from the element number which is in
+        the general data content.  */
+    }
+
+  if (first_line >= 0)
+    {
+      first_line += num_to_scroll;
+      erase_data_content (NULL);
+      delete_data_content_windows ();
+      display_registers_from_line (first_line);
+    }
 }
 
+/* See tui-regs.h.  */
+
+void
+tui_data_window::refresh_window ()
+{
+  tui_gen_win_info::refresh_window ();
+  for (auto &&win : m_regs_content)
+    win.refresh_window ();
+}
 
 /* This function check all displayed registers for changes in values,
    given a particular frame.  If the values have changed, they are
    updated with the new value and highlighted.  */
 void
-tui_check_register_values (struct frame_info *frame)
+tui_data_window::check_register_values (struct frame_info *frame)
 {
-  if (TUI_DATA_WIN != NULL
-      && TUI_DATA_WIN->generic.is_visible)
+  if (m_regs_content.empty ())
+    show_registers (m_current_group);
+  else
     {
-      if (TUI_DATA_WIN->regs_content_count <= 0 
-         && TUI_DATA_WIN->display_regs)
-       tui_show_registers (TUI_DATA_WIN->current_group);
-      else
+      for (auto &&data_item_win : m_regs_content)
        {
-         int i;
+         int was_hilighted;
 
-         for (i = 0; (i < TUI_DATA_WIN->regs_content_count); i++)
-           {
-             struct tui_data_element *data;
-             struct tui_gen_win_info *data_item_win_ptr;
-             int was_hilighted;
-
-             data_item_win_ptr = &TUI_DATA_WIN->regs_content[i]->
-                which_element.data_window;
-             data = &data_item_win_ptr->content[0]->which_element.data;
-             was_hilighted = data->highlight;
-
-              tui_get_register (frame, data,
-                                data->item_no, &data->highlight);
-
-             if (data->highlight || was_hilighted)
-               {
-                  tui_display_register (data, data_item_win_ptr);
-               }
-           }
+         was_hilighted = data_item_win.highlight;
+
+         tui_get_register (frame, &data_item_win,
+                           data_item_win.item_no,
+                           &data_item_win.highlight);
+
+         if (data_item_win.highlight || was_hilighted)
+           data_item_win.rerender ();
        }
     }
 }
 
 /* Display a register in a window.  If hilite is TRUE, then the value
    will be displayed in reverse video.  */
-static void
-tui_display_register (struct tui_data_element *data,
-                      struct tui_gen_win_info *win_info)
+void
+tui_data_item_window::rerender ()
 {
-  if (win_info->handle != NULL)
-    {
-      int i;
-
-      if (data->highlight)
-       /* We ignore the return value, casting it to void in order to avoid
-          a compiler warning.  The warning itself was introduced by a patch
-          to ncurses 5.7 dated 2009-08-29, changing this macro to expand
-          to code that causes the compiler to generate an unused-value
-          warning.  */
-       (void) wstandout (win_info->handle);
+  int i;
+
+  scrollok (handle.get (), FALSE);
+  if (highlight)
+    /* We ignore the return value, casting it to void in order to avoid
+       a compiler warning.  The warning itself was introduced by a patch
+       to ncurses 5.7 dated 2009-08-29, changing this macro to expand
+       to code that causes the compiler to generate an unused-value
+       warning.  */
+    (void) wstandout (handle.get ());
       
-      wmove (win_info->handle, 0, 0);
-      for (i = 1; i < win_info->width; i++)
-        waddch (win_info->handle, ' ');
-      wmove (win_info->handle, 0, 0);
-      if (data->content)
-        waddstr (win_info->handle, data->content);
-
-      if (data->highlight)
-       /* We ignore the return value, casting it to void in order to avoid
-          a compiler warning.  The warning itself was introduced by a patch
-          to ncurses 5.7 dated 2009-08-29, changing this macro to expand
-          to code that causes the compiler to generate an unused-value
-          warning.  */
-       (void) wstandend (win_info->handle);
-      tui_refresh_win (win_info);
+  wmove (handle.get (), 0, 0);
+  for (i = 1; i < width; i++)
+    waddch (handle.get (), ' ');
+  wmove (handle.get (), 0, 0);
+  if (content)
+    waddstr (handle.get (), content.get ());
+
+  if (highlight)
+    /* We ignore the return value, casting it to void in order to avoid
+       a compiler warning.  The warning itself was introduced by a patch
+       to ncurses 5.7 dated 2009-08-29, changing this macro to expand
+       to code that causes the compiler to generate an unused-value
+       warning.  */
+    (void) wstandend (handle.get ());
+  refresh_window ();
+}
+
+void
+tui_data_item_window::refresh_window ()
+{
+  if (handle != nullptr)
+    {
+      /* This seems to be needed because the data items are nested
+        windows, which according to the ncurses man pages aren't well
+        supported.  */
+      touchwin (handle.get ());
+      wrefresh (handle.get ());
     }
 }
 
@@ -543,14 +512,13 @@ tui_display_register (struct tui_data_element *data,
    register window is not currently being displayed.  */
 
 static struct reggroup *
-tui_reg_next (struct gdbarch *gdbarch)
+tui_reg_next (struct reggroup *current_group, struct gdbarch *gdbarch)
 {
   struct reggroup *group = NULL;
 
-  if (TUI_DATA_WIN != NULL)
+  if (current_group != NULL)
     {
-      group = TUI_DATA_WIN->current_group;
-      group = reggroup_next (gdbarch, group);
+      group = reggroup_next (gdbarch, current_group);
       if (group == NULL)
         group = reggroup_next (gdbarch, NULL);
     }
@@ -562,20 +530,34 @@ tui_reg_next (struct gdbarch *gdbarch)
    register window is not currently being displayed.  */
 
 static struct reggroup *
-tui_reg_prev (struct gdbarch *gdbarch)
+tui_reg_prev (struct reggroup *current_group, struct gdbarch *gdbarch)
 {
   struct reggroup *group = NULL;
 
-  if (TUI_DATA_WIN != NULL)
+  if (current_group != NULL)
     {
-      group = TUI_DATA_WIN->current_group;
-      group = reggroup_prev (gdbarch, group);
+      group = reggroup_prev (gdbarch, current_group);
       if (group == NULL)
        group = reggroup_prev (gdbarch, NULL);
     }
   return group;
 }
 
+/* A helper function to display the register window in the appropriate
+   way.  */
+
+static void
+tui_reg_layout ()
+{
+  enum tui_layout_type cur_layout = tui_current_layout ();
+  enum tui_layout_type new_layout;
+  if (cur_layout == SRC_COMMAND || cur_layout == SRC_DATA_COMMAND)
+    new_layout = SRC_DATA_COMMAND;
+  else
+    new_layout = DISASSEM_DATA_COMMAND;
+  tui_set_layout (new_layout);
+}
+
 /* Implement the 'tui reg' command.  Changes the register group displayed
    in the tui register window.  Displays the tui register window if it is
    not already on display.  */
@@ -596,13 +578,14 @@ tui_reg_command (const char *args, int from_tty)
       /* Make sure the register window is visible.  If not, select an
         appropriate layout.  We need to do this before trying to run the
         'next' or 'prev' commands.  */
-      if (TUI_DATA_WIN == NULL || !TUI_DATA_WIN->generic.is_visible)
-       tui_set_layout_by_name (DATA_NAME);
+      if (TUI_DATA_WIN == NULL || !TUI_DATA_WIN->is_visible ())
+       tui_reg_layout ();
 
+      struct reggroup *current_group = TUI_DATA_WIN->get_current_group ();
       if (strncmp (args, "next", len) == 0)
-       match = tui_reg_next (gdbarch);
+       match = tui_reg_next (current_group, gdbarch);
       else if (strncmp (args, "prev", len) == 0)
-       match = tui_reg_prev (gdbarch);
+       match = tui_reg_prev (current_group, gdbarch);
 
       /* This loop matches on the initial part of a register group
         name.  If this initial part in ARGS matches only one register
@@ -622,7 +605,7 @@ tui_reg_command (const char *args, int from_tty)
       if (match == NULL)
        error (_("unknown register group '%s'"), args);
 
-      tui_show_registers (match);
+      TUI_DATA_WIN->show_registers (match);
     }
   else
     {
@@ -679,63 +662,3 @@ _initialize_tui_regs (void)
 TUI command to control the register window."), tuicmd);
   set_cmd_completer (cmd, tui_reggroup_completer);
 }
-
-
-/*****************************************
-** STATIC LOCAL FUNCTIONS                 **
-******************************************/
-
-/* Get the register from the frame and return a printable
-   representation of it.  */
-
-static char *
-tui_register_format (struct frame_info *frame, int regnum)
-{
-  struct gdbarch *gdbarch = get_frame_arch (frame);
-
-  string_file stream;
-
-  scoped_restore save_pagination
-    = make_scoped_restore (&pagination_enabled, 0);
-  scoped_restore save_stdout
-    = make_scoped_restore (&gdb_stdout, &stream);
-
-  gdbarch_print_registers_info (gdbarch, &stream, frame, regnum, 1);
-
-  /* Remove the possible \n.  */
-  std::string &str = stream.string ();
-  if (!str.empty () && str.back () == '\n')
-    str.resize (str.size () - 1);
-
-  /* Expand tabs into spaces, since ncurses on MS-Windows doesn't.  */
-  return tui_expand_tabs (str.c_str (), 0);
-}
-
-/* Get the register value from the given frame and format it for the
-   display.  When changep is set, check if the new register value has
-   changed with respect to the previous call.  */
-static enum tui_status
-tui_get_register (struct frame_info *frame,
-                  struct tui_data_element *data, 
-                 int regnum, int *changedp)
-{
-  enum tui_status ret = TUI_FAILURE;
-
-  if (changedp)
-    *changedp = FALSE;
-  if (target_has_registers)
-    {
-      char *prev_content = data->content;
-
-      data->content = tui_register_format (frame, regnum);
-
-      if (changedp != NULL
-         && strcmp (prev_content, data->content) != 0)
-       *changedp = 1;
-
-      xfree (prev_content);
-
-      ret = TUI_SUCCESS;
-    }
-  return ret;
-}
This page took 0.057254 seconds and 4 git commands to generate.