aboutsummaryrefslogtreecommitdiff
path: root/gdb/tui/tui-layout.c
diff options
context:
space:
mode:
authorAndrew Burgess <aburgess@redhat.com>2022-02-01 14:09:26 +0000
committerAndrew Burgess <aburgess@redhat.com>2022-04-03 15:31:47 +0100
commitb45b7407cdd07900ecc28a36710603edc96ade71 (patch)
tree51b0492155ad6ce5274736e1c3395292e02e0985 /gdb/tui/tui-layout.c
parentb1ac644a05245de73b84c23fa6a24eb039a727d7 (diff)
downloadgdb-b45b7407cdd07900ecc28a36710603edc96ade71.zip
gdb-b45b7407cdd07900ecc28a36710603edc96ade71.tar.gz
gdb-b45b7407cdd07900ecc28a36710603edc96ade71.tar.bz2
gdb/tui: relax restrictions on window max height and width
This commit removes some arbitrary adjustments made in tui_cmd_window::max_height, tui_win_info::max_height, and tui_win_info::max_width. These member functions all subtract some constant from the theoretical maximum height or width. I've looked back through the history a little and can see no real reason why these adjustments should be needed, with these adjustments removed all the existing tui tests still pass. However, retaining these restrictions causes some bugs, consider: (gdb) tui new-layout hsrc {-horizontal src 1 cmd 1} 1 When this layout is selected with current master, gdb will leave a 4 line gap at the bottom of the terminal. The problem is that the maximum height is restricted, for the cmd window, to 4 less than the terminal height. By removing this restriction gdb is able to size the windows to the complete terminal height, and the layout is done correctly. This 4 line restriction is also what prevents this layout from working correctly: (gdb) tui new-layout conly cmd 1 Previously, this layout would present a cmd window only, but there would be a 4 line gap at the bottom of the terminal. This issue was mentioned in an earlier commit in this series (when a different bug was fixed), but with this commit, the above layout now correctly fills the terminal. The associated test is updated. After removing the adjustment in tui_cmd_window::max_height, the implementation is now the same as the implementation in the parent class tui_win_info, so I've completely removed the max_height call from tui_cmd_window.
Diffstat (limited to 'gdb/tui/tui-layout.c')
0 files changed, 0 insertions, 0 deletions