From 89df7f9056d72418720ac655e1ad97decc471da9 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Sun, 14 Jul 2019 19:45:55 -0600 Subject: [PATCH] Private data members in tui_data_window This changes tui_data_window so that the data members are private. This required the addition of a simple accessor method in one case. gdb/ChangeLog 2019-08-30 Tom Tromey * tui/tui-regs.h (struct tui_data_window) : Move later. Now private. : New method. * tui/tui-regs.c (tui_reg_command): Update. * tui/tui-layout.c (tui_set_layout): Update. --- gdb/ChangeLog | 8 ++++++++ gdb/tui/tui-layout.c | 2 +- gdb/tui/tui-regs.c | 2 +- gdb/tui/tui-regs.h | 15 ++++++++++----- 4 files changed, 20 insertions(+), 7 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index b95be65..2a2c08c 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,13 @@ 2019-08-30 Tom Tromey + * tui/tui-regs.h (struct tui_data_window) : Move later. Now private. + : New method. + * tui/tui-regs.c (tui_reg_command): Update. + * tui/tui-layout.c (tui_set_layout): Update. + +2019-08-30 Tom Tromey + * tui/tui-regs.c (tui_data_window::display_registers_from_line) (tui_data_window::rerender): Don't call check_and_display_highlight_if_needed. diff --git a/gdb/tui/tui-layout.c b/gdb/tui/tui-layout.c index 01d50e4..387a4f5 100644 --- a/gdb/tui/tui-layout.c +++ b/gdb/tui/tui-layout.c @@ -189,7 +189,7 @@ tui_set_layout (enum tui_layout_type layout_type) tui_update_source_windows_with_addr (gdbarch, addr); if (new_layout == SRC_DATA_COMMAND || new_layout == DISASSEM_DATA_COMMAND) - TUI_DATA_WIN->show_registers (TUI_DATA_WIN->current_group); + TUI_DATA_WIN->show_registers (TUI_DATA_WIN->get_current_group ()); } } diff --git a/gdb/tui/tui-regs.c b/gdb/tui/tui-regs.c index 4a09a59..98096e2 100644 --- a/gdb/tui/tui-regs.c +++ b/gdb/tui/tui-regs.c @@ -622,7 +622,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) diff --git a/gdb/tui/tui-regs.h b/gdb/tui/tui-regs.h index fa76cae..2606c39 100644 --- a/gdb/tui/tui-regs.h +++ b/gdb/tui/tui-regs.h @@ -63,15 +63,15 @@ struct tui_data_window : public tui_win_info return DATA_NAME; } - /* Windows that are used to display registers. */ - std::vector regs_content; - int regs_column_count = 0; - struct reggroup *current_group = nullptr; - void check_register_values (struct frame_info *frame); void show_registers (struct reggroup *group); + struct reggroup *get_current_group () const + { + return current_group; + } + protected: void do_scroll_vertical (int num_to_scroll) override; @@ -126,6 +126,11 @@ private: void delete_data_content_windows (); void erase_data_content (const char *prompt); + + /* Windows that are used to display registers. */ + std::vector regs_content; + int regs_column_count = 0; + struct reggroup *current_group = nullptr; }; #endif /* TUI_TUI_REGS_H */ -- 2.7.4