clear_detail can only be called on TUI source windows
authorTom Tromey <tom@tromey.com>
Fri, 5 Jul 2019 18:00:06 +0000 (12:00 -0600)
committerTom Tromey <tom@tromey.com>
Tue, 13 Aug 2019 20:52:09 +0000 (14:52 -0600)
The clear_detail method can only be called on source windows, so
remove definitions from the base of the class hierarchy, leaving only
a single non-virtual method.

gdb/ChangeLog
2019-08-13  Tom Tromey  <tom@tromey.com>

* tui/tui-winsource.h (struct tui_source_window_base)
<clear_detail>: No longer "override".
* tui/tui-regs.h (struct tui_data_window) <clear_detail>: Remove.
* tui/tui-regs.c (tui_data_window::clear_detail): Remove.
* tui/tui-data.h (struct tui_win_info) <clear_detail>: Remove.
* tui/tui-command.h (struct tui_cmd_window) <clear_detail>:
Remove.
* tui/tui-command.c (tui_cmd_window::clear_detail): Remove.

gdb/ChangeLog
gdb/tui/tui-command.c
gdb/tui/tui-command.h
gdb/tui/tui-data.h
gdb/tui/tui-regs.c
gdb/tui/tui-regs.h
gdb/tui/tui-winsource.h

index 6c4208f..58a32a2 100644 (file)
@@ -1,3 +1,14 @@
+2019-08-13  Tom Tromey  <tom@tromey.com>
+
+       * tui/tui-winsource.h (struct tui_source_window_base)
+       <clear_detail>: No longer "override".
+       * tui/tui-regs.h (struct tui_data_window) <clear_detail>: Remove.
+       * tui/tui-regs.c (tui_data_window::clear_detail): Remove.
+       * tui/tui-data.h (struct tui_win_info) <clear_detail>: Remove.
+       * tui/tui-command.h (struct tui_cmd_window) <clear_detail>:
+       Remove.
+       * tui/tui-command.c (tui_cmd_window::clear_detail): Remove.
+
 2019-08-13  Tom Tromey  <tromey@adacore.com>
 
        * tracepoint.c: Don't include readline.h or history.h.
index 5af3484..034abd5 100644 (file)
 /* See tui-command.h.  */
 
 void
-tui_cmd_window::clear_detail ()
-{
-  wmove (handle, 0, 0);
-}
-
-/* See tui-command.h.  */
-
-void
 tui_cmd_window::do_make_visible_with_new_height ()
 {
 #ifdef HAVE_WRESIZE
index d2a8ab1..af80b14 100644 (file)
@@ -35,8 +35,6 @@ struct tui_cmd_window : public tui_win_info
 
   DISABLE_COPY_AND_ASSIGN (tui_cmd_window);
 
-  void clear_detail () override;
-
   void make_visible (bool visible) override
   {
   }
index 214f728..5a7c487 100644 (file)
@@ -249,9 +249,6 @@ public:
   {
   }
 
-  /* Clear the pertinent detail in the window.  */
-  virtual void clear_detail () = 0;
-
   /* Called after all the TUI windows are refreshed, to let this
      window have a chance to update itself further.  */
   virtual void refresh_all ()
index c13a5b1..2765462 100644 (file)
@@ -521,16 +521,6 @@ tui_data_window::do_scroll_vertical (int num_to_scroll)
 /* See tui-regs.h.  */
 
 void
-tui_data_window::clear_detail ()
-{
-  regs_content.clear ();
-  regs_column_count = 1;
-  display_regs = false;
-}
-
-/* See tui-regs.h.  */
-
-void
 tui_data_window::set_new_height (int height)
 {
   /* Delete all data item windows.  */
index 0646729..b2abfc2 100644 (file)
@@ -53,7 +53,6 @@ struct tui_data_window : public tui_win_info
 
   DISABLE_COPY_AND_ASSIGN (tui_data_window);
 
-  void clear_detail () override;
   void refresh_all () override;
 
   void set_new_height (int height) override;
index ec44d1d..9d7f0a7 100644 (file)
@@ -67,7 +67,7 @@ protected:
 
 public:
 
-  void clear_detail () override;
+  void clear_detail ();
 
   void make_visible (bool visible) override;
   void refresh_window () override;