Remove tui_display_data_from
authorTom Tromey <tom@tromey.com>
Wed, 26 Jun 2019 22:00:05 +0000 (16:00 -0600)
committerTom Tromey <tom@tromey.com>
Wed, 17 Jul 2019 18:18:57 +0000 (12:18 -0600)
tui_display_data_from is only called from a single place.  Inlining it
there lets us remove some uses of the TUI_DATA_WIN global.

gdb/ChangeLog
2019-07-17  Tom Tromey  <tom@tromey.com>

* tui/tui-windata.h (tui_display_data_from): Don't declare.
* tui/tui-windata.c (tui_display_data_from): Remove.
(tui_data_window::refresh_all): Update.

gdb/ChangeLog
gdb/tui/tui-windata.c
gdb/tui/tui-windata.h

index d7df8fd34567e0adb7e4d858ad295563574c4ea8..3daafb35d9d76ec0867ec6cfd6e287b1c9c11311 100644 (file)
@@ -1,3 +1,9 @@
+2019-07-17  Tom Tromey  <tom@tromey.com>
+
+       * tui/tui-windata.h (tui_display_data_from): Don't declare.
+       * tui/tui-windata.c (tui_display_data_from): Remove.
+       (tui_data_window::refresh_all): Update.
+
 2019-07-17  Tom Tromey  <tom@tromey.com>
 
        * tui/tui-windata.h (tui_display_data_from_line): Don't declare.
index 4f18d73aec28949cf892dc0677c7e0226b7873d0..722d0a5e26e29bf04e3a41da97f103cfee79c764 100644 (file)
@@ -111,28 +111,6 @@ tui_display_all_data (void)
 }
 
 
-/* Display data starting at element element_no.  */
-void
-tui_display_data_from (int element_no, int reuse_windows)
-{
-  int first_line = (-1);
-
-  if (element_no < TUI_DATA_WIN->regs_content.size ())
-    first_line = TUI_DATA_WIN->line_from_reg_element_no (element_no);
-  else
-    { /* Calculate the first_line from the element number.  */
-    }
-
-  if (first_line >= 0)
-    {
-      tui_erase_data_content (NULL);
-      if (!reuse_windows)
-       tui_delete_data_content_windows ();
-      tui_display_registers_from_line (first_line);
-    }
-}
-
-
 /* Function to redisplay the contents of the data window.  */
 void
 tui_data_window::refresh_all ()
@@ -143,7 +121,18 @@ tui_data_window::refresh_all ()
       int first_element = first_data_item_displayed ();
 
       if (first_element >= 0)  /* Re-use existing windows.  */
-       tui_display_data_from (first_element, TRUE);
+       {
+         int first_line = (-1);
+
+         if (first_element < regs_content.size ())
+           first_line = line_from_reg_element_no (first_element);
+
+         if (first_line >= 0)
+           {
+             tui_erase_data_content (NULL);
+             tui_display_registers_from_line (first_line);
+           }
+       }
     }
 }
 
index fd8e6257da76b052e78e6f7c6d550139c6eef803..9970cfae54701f0f667c0c05263e55e2684f8371 100644 (file)
@@ -28,6 +28,5 @@ extern void tui_erase_data_content (const char *);
 extern void tui_display_all_data (void);
 extern void tui_delete_data_content_windows (void);
 extern void tui_refresh_data_win (void);
-extern void tui_display_data_from (int, int);
 
 #endif /* TUI_TUI_WINDATA_H */