Turn tui_show_exec_info_content into a method
authorTom Tromey <tom@tromey.com>
Fri, 5 Jul 2019 23:35:13 +0000 (17:35 -0600)
committerTom Tromey <tom@tromey.com>
Tue, 13 Aug 2019 20:52:09 +0000 (14:52 -0600)
This changes tui_show_exec_info_content to be a method on
tui_source_window_base.  As it is only called by other methods on this
class, it can be private.

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

* tui/tui-winsource.h (struct tui_source_window_base)
<show_exec_info_content>: Declare.
(tui_show_exec_info_content): Don't declare.
* tui/tui-winsource.c
(tui_source_window_base::show_exec_info_content): Rename from
tui_show_exec_info_content.
(tui_source_window_base::update_exec_info): Update.

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

index 750ce86..a739724 100644 (file)
@@ -1,5 +1,15 @@
 2019-08-13  Tom Tromey  <tom@tromey.com>
 
+       * tui/tui-winsource.h (struct tui_source_window_base)
+       <show_exec_info_content>: Declare.
+       (tui_show_exec_info_content): Don't declare.
+       * tui/tui-winsource.c
+       (tui_source_window_base::show_exec_info_content): Rename from
+       tui_show_exec_info_content.
+       (tui_source_window_base::update_exec_info): Update.
+
+2019-08-13  Tom Tromey  <tom@tromey.com>
+
        * tui/tui-data.h (enum tui_bp_flag, tui_bp_flags, struct tui_source_element)
        (TUI_BP_HIT_POS, TUI_BP_BREAK_POS, TUI_EXEC_POS)
        (TUI_EXECINFO_SIZE, tui_exec_info_content): Move ...
index 53113bd..c99e207 100644 (file)
@@ -641,17 +641,17 @@ tui_source_window_base::set_exec_info_content ()
 
 
 void
-tui_show_exec_info_content (struct tui_source_window_base *win_info)
+tui_source_window_base::show_exec_info_content ()
 {
-  struct tui_exec_info_window *exec_info = win_info->execution_info;
-  const tui_exec_info_content *content = exec_info->get_content ();
+  struct tui_exec_info_window *exec_info = execution_info;
+  const tui_exec_info_content *exec_content = exec_info->get_content ();
 
   werase (exec_info->handle);
-  for (int cur_line = 1; cur_line <= win_info->content.size (); cur_line++)
+  for (int cur_line = 1; cur_line <= content.size (); cur_line++)
     mvwaddstr (exec_info->handle,
               cur_line,
               0,
-              content[cur_line - 1]);
+              exec_content[cur_line - 1]);
   exec_info->refresh_window ();
 }
 
@@ -676,7 +676,7 @@ void
 tui_source_window_base::update_exec_info ()
 {
   set_exec_info_content ();
-  tui_show_exec_info_content (this);
+  show_exec_info_content ();
 }
 
 void
index 71201e5..c902ee0 100644 (file)
@@ -150,6 +150,9 @@ public:
   struct gdbarch *gdbarch = nullptr;
 
   std::vector<tui_source_element> content;
+
+private:
+  void show_exec_info_content ();
 };
 
 /* Update the execution windows to show the active breakpoints.  This
@@ -185,7 +188,6 @@ extern void tui_update_source_windows_with_line (struct symtab *,
                                                 int);
 extern void tui_clear_source_content (struct tui_source_window_base *);
 extern void tui_erase_source_content (struct tui_source_window_base *);
-extern void tui_show_exec_info_content (struct tui_source_window_base *);
 extern void tui_erase_exec_info_content (struct tui_source_window_base *);
 extern void tui_clear_exec_info_content (struct tui_source_window_base *);