Remove buggy xterm workaround in tui_dispatch_ctrl_char()
authorPatrick Palka <patrick@parcs.ath.cx>
Wed, 13 May 2015 20:26:48 +0000 (16:26 -0400)
committerPatrick Palka <patrick@parcs.ath.cx>
Thu, 14 May 2015 12:18:06 +0000 (08:18 -0400)
commite3555239e04fa6dba2a165b3b52598a880474a22
tree157984015f197084d85dd271980d254c1278a3c6
parentfdc3b1b1ef461fb81a53c9ce95117e5f134fc828
Remove buggy xterm workaround in tui_dispatch_ctrl_char()

The function tui_dispatch_ctrl_char() has an old workaround (from 1999)
for buggy terminals and/or ncurses library that don't return page
up/down keys as single characters.  Because the workaround is so old, I
think the bug it is targetting is no longer relevant anymore.

But more importantly, the workaround is itself buggy: it 1) performs a
blocking call to wgetch() and 2) if the key returned by wgetch() does
not make up a relevant key sequence it throws away the input instead of
pushing it back via ungetch().  And indeed the workaround breaks Alt-key
sequences under TERM=xterm because of bug #2.

So this patch removes the buggy workaround and tidies up the function
accordingly.

I personally tested this change on a recent xterm (with TERM=xterm) in
Fedora 20 and had no problems with having ncurses properly interpret
page up/down keys.  And Alt-key sequences now work when TERM=xterm too.

gdb/ChangeLog:

* tui/tui-command.c: Remove include of <ctype.h>.
(tui_dispatch_ctrl_char): Remove workaround for xterm terminals.
gdb/ChangeLog
gdb/tui/tui-command.c