diff options
author | Tom Tromey <tom@tromey.com> | 2019-07-18 14:38:39 -0600 |
---|---|---|
committer | Tom Tromey <tom@tromey.com> | 2019-08-30 12:57:10 -0600 |
commit | d6a00eba2accffec92a5974c2ad1f79612a6679e (patch) | |
tree | 2319ac6eea7c7d8adc469d7f2ada9ea0abc2b17e /readline/util.c | |
parent | 55b2657bdc4f9494c13f5b6f69dd7f67d72275c0 (diff) | |
download | gdb-d6a00eba2accffec92a5974c2ad1f79612a6679e.zip gdb-d6a00eba2accffec92a5974c2ad1f79612a6679e.tar.gz gdb-d6a00eba2accffec92a5974c2ad1f79612a6679e.tar.bz2 |
Remove tui_win_info::refresh_all
The TUI has two duplicate "re-render this window" methods, "rerender"
and "refresh_all". They differ only slightly in semantics, so I
wanted to see if they could be unified.
After looking into this, I decided that refresh_all was not needed.
There are 4 calls to tui_refresh_all_win (the only caller of this
method):
1. tui_enable. This sets the layout, which renders the windows.
2. tui_cont_sig. Here, I think it's sufficient to simply redraw the
current window contents from the curses backing store, because gdb
state didn't change while it was suspended
3. tui_dispatch_ctrl_char. This is the C-l handler, and here it's
explicitly enough to just refresh the screen (as above).
4. tui_refresh_all_command. This is the command equivalent of C-l.
So, this patch removes this method entirely and simplifies
tui_refresh_all_win.
gdb/ChangeLog
2019-08-30 Tom Tromey <tom@tromey.com>
* tui/tui-winsource.h (struct tui_source_window_base)
<refresh_all>: Don't declare.
* tui/tui-winsource.c (tui_source_window_base::refresh_all):
Remove.
* tui/tui-win.c (tui_refresh_all_win): Don't call refresh_all or
tui_show_locator_content.
* tui/tui-regs.h (struct tui_data_window) <refresh_all>: Don't
declare.
* tui/tui-regs.c (tui_data_window::refresh_all): Remove.
* tui/tui-data.h (struct tui_win_info) <refresh_all>: Don't
declare.
Diffstat (limited to 'readline/util.c')
0 files changed, 0 insertions, 0 deletions