gdb/tui: convert if/error to an assert
authorAndrew Burgess <aburgess@redhat.com>
Thu, 22 Dec 2022 12:43:38 +0000 (12:43 +0000)
committerAndrew Burgess <aburgess@redhat.com>
Wed, 25 Jan 2023 10:51:03 +0000 (10:51 +0000)
commit6789344ab22f7fbce94a31297b994f85685b90c6
treee600cd4c51b70a8f98caff1b38b9610b750fcde5
parent24f3aded1d42f515527e2de7e8e9e26f0b77c932
gdb/tui: convert if/error to an assert

While working on the previous commit, I realised that there was an
error in tui_set_focus_command that could never be triggered.

Since the big tui rewrite (adding dynamic layouts) it is no longer
true that there is a tui_win_info object for every window at all
times.  We now only create a tui_win_info object for a particular
window, when the window is part of the current layout.  As a result,
if we have a tui_win_info pointer, then the window must be visible
inside tui_set_focus_command (this function calls tui_enable as its
first action, which makes the current layout visible).

The gdb.tui/tui-focus.exp test script exercises this area of code, and
doesn't trigger the assert, nor do any of our other existing tui
tests.
gdb/tui/tui-win.c