diff options
author | Tom Tromey <tom@tromey.com> | 2019-10-26 16:56:27 -0600 |
---|---|---|
committer | Tom Tromey <tom@tromey.com> | 2019-11-19 13:27:25 -0700 |
commit | 9f6ad286efc0edf7d0cdb8a4126a23d38a267f8d (patch) | |
tree | ffc49d645384be569e25d409d9abfd659eb74d00 /gdb/tui/tui-data.c | |
parent | 435d3d88360d1ad9aac798f6bff7d7b5363efe3e (diff) | |
download | fsf-binutils-gdb-9f6ad286efc0edf7d0cdb8a4126a23d38a267f8d.zip fsf-binutils-gdb-9f6ad286efc0edf7d0cdb8a4126a23d38a267f8d.tar.gz fsf-binutils-gdb-9f6ad286efc0edf7d0cdb8a4126a23d38a267f8d.tar.bz2 |
Fix the "winheight" command
The "winheight" command is broken. I probably broke it in one of my
TUI refactoring patches, though I didn't track down exactly which one.
The bug is that the code does:
*buf_ptr = '\0';
... but then never advances buf_ptr past this point, so no window name
is seen.
This patch refactors the code a bit so that a copy of the argument
string is not needed, also fixing the bug.
A new test case is included.
gdb/ChangeLog
2019-11-19 Tom Tromey <tom@tromey.com>
* tui/tui-win.c (tui_partial_win_by_name): Move from tui-data.c.
Now static. Change type of "name".
(tui_set_win_height_command): Don't copy "arg".
* tui/tui-data.h (tui_partial_win_by_name): Don't declare.
* tui/tui-data.c (tui_partial_win_by_name): Move to tui-win.c.
gdb/testsuite/ChangeLog
2019-11-19 Tom Tromey <tom@tromey.com>
* gdb.tui/winheight.exp: New file.
Change-Id: I0871e93777a70036dbec9c9543f862f42e3a81e5
Diffstat (limited to 'gdb/tui/tui-data.c')
-rw-r--r-- | gdb/tui/tui-data.c | 19 |
1 files changed, 0 insertions, 19 deletions
diff --git a/gdb/tui/tui-data.c b/gdb/tui/tui-data.c index 522bb9a..8f79658 100644 --- a/gdb/tui/tui-data.c +++ b/gdb/tui/tui-data.c @@ -158,25 +158,6 @@ tui_prev_win (struct tui_win_info *cur_win) } -/* Answer the window represented by name. */ -struct tui_win_info * -tui_partial_win_by_name (const char *name) -{ - if (name != NULL) - { - for (tui_win_info *item : all_tui_windows ()) - { - const char *cur_name = item->name (); - - if (strlen (name) <= strlen (cur_name) - && startswith (cur_name, name)) - return item; - } - } - - return NULL; -} - /* See tui-data.h. */ void |