Simplify register display
[external/binutils.git] / gdb / tui / tui-regs.c
index 8adfb15..f62ba06 100644 (file)
@@ -42,8 +42,6 @@
 
 #include "gdb_curses.h"
 
-static void tui_display_register (struct tui_data_item_window *data);
-
 /* Get the register from the frame and return a printable
    representation of it.  */
 
@@ -98,14 +96,9 @@ tui_get_register (struct frame_info *frame,
 int
 tui_data_window::last_regs_line_no () const
 {
-  int num_lines = (-1);
-
-  if (!regs_content.empty ())
-    {
-      num_lines = regs_content.size () / regs_column_count;
-      if (regs_content.size () % regs_column_count)
-       num_lines++;
-    }
+  int num_lines = regs_content.size () / regs_column_count;
+  if (regs_content.size () % regs_column_count)
+    num_lines++;
   return num_lines;
 }
 
@@ -152,10 +145,6 @@ tui_data_window::show_registers (struct reggroup *group)
   if (group == 0)
     group = general_reggroup;
 
-  /* Say that registers should be displayed, even if there is a
-     problem.  */
-  display_regs = true;
-
   if (target_has_registers && target_has_stack && target_has_memory)
     {
       show_register_group (group, get_selected_frame (NULL),
@@ -163,12 +152,9 @@ tui_data_window::show_registers (struct reggroup *group)
 
       /* Clear all notation of changed values.  */
       for (auto &&data_item_win : regs_content)
-       {
-         if (data_item_win != nullptr)
-           data_item_win->highlight = false;
-       }
+       data_item_win.highlight = false;
       current_group = group;
-      display_all_data ();
+      rerender ();
     }
   else
     {
@@ -192,8 +178,7 @@ tui_data_window::show_register_group (struct reggroup *group,
   int regnum, pos;
 
   /* Make a new title showing which group we display.  */
-  xfree (title);
-  title = xstrprintf ("Register group: %s", reggroup_name (group));
+  title = string_printf ("Register group: %s", reggroup_name (group));
 
   /* See how many registers must be displayed.  */
   nr_regs = 0;
@@ -214,16 +199,7 @@ tui_data_window::show_register_group (struct reggroup *group,
       nr_regs++;
     }
 
-  if (!refresh_values_only)
-    regs_content.clear ();
-
-  if (nr_regs < regs_content.size ())
-    regs_content.resize (nr_regs);
-  else
-    {
-      for (int i = regs_content.size (); i < nr_regs; ++i)
-       regs_content.emplace_back (new tui_data_item_window ());
-    }
+  regs_content.resize (nr_regs);
 
   /* Now set the register names and values.  */
   pos = 0;
@@ -242,7 +218,7 @@ tui_data_window::show_register_group (struct reggroup *group,
       if (name == 0 || *name == '\0')
        continue;
 
-      data_item_win = regs_content[pos].get ();
+      data_item_win = &regs_content[pos];
       if (data_item_win)
        {
          if (!refresh_values_only)
@@ -262,74 +238,46 @@ tui_data_window::show_register_group (struct reggroup *group,
 void
 tui_data_window::display_registers_from (int start_element_no)
 {
-  if (!regs_content.empty ())
+  int j, item_win_width, cur_y;
+
+  int max_len = 0;
+  for (auto &&data_item_win : regs_content)
     {
-      int j, item_win_width, cur_y;
+      const char *p;
+      int len;
 
-      int max_len = 0;
-      for (auto &&data_item_win : regs_content)
-        {
-          const char *p;
-          int len;
-
-          len = 0;
-          p = data_item_win->content.get ();
-          if (p != 0)
-           len = strlen (p);
-
-          if (len > max_len)
-            max_len = len;
-        }
-      item_win_width = max_len + 1;
-      int i = start_element_no;
-
-      regs_column_count = (width - 2) / item_win_width;
-      if (regs_column_count == 0)
-        regs_column_count = 1;
-      item_win_width = (width - 2) / regs_column_count;
-
-      /* Now create each data "sub" window, and write the display into
-        it.  */
-      cur_y = 1;
-      while (i < regs_content.size ()
-            && cur_y <= viewport_height)
+      len = 0;
+      p = data_item_win.content.get ();
+      if (p != 0)
+       len = strlen (p);
+
+      if (len > max_len)
+       max_len = len;
+    }
+  item_win_width = max_len + 1;
+  int i = start_element_no;
+
+  regs_column_count = (width - 2) / item_win_width;
+  if (regs_column_count == 0)
+    regs_column_count = 1;
+  item_win_width = (width - 2) / regs_column_count;
+
+  /* Now create each data "sub" window, and write the display into
+     it.  */
+  cur_y = 1;
+  while (i < regs_content.size ()
+        && cur_y <= viewport_height)
+    {
+      for (j = 0;
+          j < regs_column_count && i < regs_content.size ();
+          j++)
        {
-         for (j = 0;
-              j < regs_column_count && i < regs_content.size ();
-              j++)
-           {
-             struct tui_data_item_window *data_item_win;
-
-             /* Create the window if necessary.  */
-             data_item_win = regs_content[i].get ();
-              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);
-                  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_item_win);
-             i++;              /* Next register.  */
-           }
-         cur_y++;              /* Next row.  */
+         /* Create the window if necessary.  */
+         regs_content[i].resize (1, item_win_width,
+                                 (item_win_width * j) + 1, cur_y);
+         i++;          /* Next register.  */
        }
+      cur_y++;         /* Next row.  */
     }
 }
 
@@ -339,27 +287,24 @@ void
 tui_data_window::display_reg_element_at_line (int start_element_no,
                                              int start_line_no)
 {
-  if (!regs_content.empty ())
-    {
-      int element_no = start_element_no;
+  int element_no = start_element_no;
 
-      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);
+  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);
 }
 
 /* See tui-regs.h.  */
@@ -367,35 +312,29 @@ tui_data_window::display_reg_element_at_line (int start_element_no,
 int
 tui_data_window::display_registers_from_line (int line_no)
 {
-  check_and_display_highlight_if_needed ();
-  if (!regs_content.empty ())
-    {
-      int element_no;
+  int element_no;
 
-      if (line_no < 0)
-       line_no = 0;
-      else
+  if (line_no < 0)
+    line_no = 0;
+  else
+    {
+      /* Make sure that we don't display off the end of the
+        registers.  */
+      if (line_no >= last_regs_line_no ())
        {
-         /* Make sure that we don't display off the end of the
-            registers.  */
-         if (line_no >= last_regs_line_no ())
-           {
-             line_no = line_from_reg_element_no (regs_content.size () - 1);
-             if (line_no < 0)
-               line_no = 0;
-           }
+         line_no = line_from_reg_element_no (regs_content.size () - 1);
+         if (line_no < 0)
+           line_no = 0;
        }
-
-      element_no = first_reg_element_no_inline (line_no);
-      if (element_no < regs_content.size ())
-       display_reg_element_at_line (element_no, line_no);
-      else
-       line_no = (-1);
-
-      return line_no;
     }
 
-  return (-1);                 /* Nothing was displayed.  */
+  element_no = first_reg_element_no_inline (line_no);
+  if (element_no < regs_content.size ())
+    display_reg_element_at_line (element_no, line_no);
+  else
+    line_no = (-1);
+
+  return line_no;
 }
 
 
@@ -408,7 +347,7 @@ tui_data_window::first_data_item_displayed ()
     {
       struct tui_gen_win_info *data_item_win;
 
-      data_item_win = regs_content[i].get ();
+      data_item_win = &regs_content[i];
       if (data_item_win->is_visible ())
        return i;
     }
@@ -423,8 +362,8 @@ tui_data_window::delete_data_content_windows ()
 {
   for (auto &&win : regs_content)
     {
-      tui_delete_win (win->handle);
-      win->handle = NULL;
+      tui_delete_win (win.handle);
+      win.handle = NULL;
     }
 }
 
@@ -451,15 +390,14 @@ tui_data_window::erase_data_content (const char *prompt)
 /* See tui-regs.h.  */
 
 void
-tui_data_window::display_all_data ()
+tui_data_window::rerender ()
 {
   if (regs_content.empty ())
-    erase_data_content (NO_DATA_STRING);
+    erase_data_content (_("[ Register Values Unavailable ]"));
   else
     {
       erase_data_content (NULL);
       delete_data_content_windows ();
-      check_and_display_highlight_if_needed ();
       display_registers_from (0);
     }
 }
@@ -482,10 +420,7 @@ tui_data_window::refresh_all ()
            first_line = line_from_reg_element_no (first_element);
 
          if (first_line >= 0)
-           {
-             erase_data_content (NULL);
-             display_registers_from_line (first_line);
-           }
+           display_registers_from_line (first_line);
        }
     }
 }
@@ -518,28 +453,11 @@ tui_data_window::do_scroll_vertical (int num_to_scroll)
 /* See tui-regs.h.  */
 
 void
-tui_data_window::rerender ()
-{
-  /* Delete all data item windows.  */
-  for (auto &&win : regs_content)
-    {
-      tui_delete_win (win->handle);
-      win->handle = NULL;
-    }
-  display_all_data ();
-}
-
-/* See tui-regs.h.  */
-
-void
 tui_data_window::refresh_window ()
 {
   tui_gen_win_info::refresh_window ();
   for (auto &&win : regs_content)
-    {
-      if (win != NULL)
-       win->refresh_window ();
-    }
+    win.refresh_window ();
 }
 
 /* This function check all displayed registers for changes in values,
@@ -548,58 +466,69 @@ tui_data_window::refresh_window ()
 void
 tui_data_window::check_register_values (struct frame_info *frame)
 {
-  if (regs_content.empty () && display_regs)
+  if (regs_content.empty ())
     show_registers (current_group);
   else
     {
-      for (auto &&data_item_win_ptr : regs_content)
+      for (auto &&data_item_win : regs_content)
        {
          int was_hilighted;
 
-         was_hilighted = data_item_win_ptr->highlight;
+         was_hilighted = data_item_win.highlight;
 
-         tui_get_register (frame, data_item_win_ptr.get (),
-                           data_item_win_ptr->item_no,
-                           &data_item_win_ptr->highlight);
+         tui_get_register (frame, &data_item_win,
+                           data_item_win.item_no,
+                           &data_item_win.highlight);
 
-         if (data_item_win_ptr->highlight || was_hilighted)
-           tui_display_register (data_item_win_ptr.get ());
+         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_item_window *data)
+void
+tui_data_item_window::rerender ()
 {
-  if (data->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 (data->handle);
+  int i;
+
+  scrollok (handle, 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);
       
-      wmove (data->handle, 0, 0);
-      for (i = 1; i < data->width; i++)
-        waddch (data->handle, ' ');
-      wmove (data->handle, 0, 0);
-      if (data->content)
-        waddstr (data->handle, data->content.get ());
-
-      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 (data->handle);
-      data->refresh_window ();
+  wmove (handle, 0, 0);
+  for (i = 1; i < width; i++)
+    waddch (handle, ' ');
+  wmove (handle, 0, 0);
+  if (content)
+    waddstr (handle, 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);
+  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);
+      wrefresh (handle);
     }
 }
 
@@ -677,7 +606,7 @@ tui_reg_command (const char *args, int from_tty)
       if (TUI_DATA_WIN == NULL || !TUI_DATA_WIN->is_visible ())
        tui_reg_layout ();
 
-      struct reggroup *current_group = TUI_DATA_WIN->current_group;
+      struct reggroup *current_group = TUI_DATA_WIN->get_current_group ();
       if (strncmp (args, "next", len) == 0)
        match = tui_reg_next (current_group, gdbarch);
       else if (strncmp (args, "prev", len) == 0)