d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix vt100 escape sequence for showing and hiding the cursor in syscons.

Browse Source 
It should toggle between 2 states, but it used a cut-down version of
support for a related 3-state syscons escape sequence and inherited
bugs from that.  The usual misbehaviour was that hiding and showing
the cursor reset it to a global default.

Support for the 3-state sequence remains broken by aliasing to the 2-state
sequence.  This works better but incompatibly for the 2 cases that it
supports.
This commit is contained in:
Bruce Evans 2017-08-18 12:45:00 +00:00
parent a54b74c7f3
commit e4501d816b
1 changed files with 6 additions and 7 deletions

13
sys/dev/syscons/scterm-teken.c

@ -674,6 +674,7 @@ static void
scteken_param(void *arg, int cmd, unsigned int value)
{
scr_stat *scp = arg;
int flags;
switch (cmd) {
case TP_SETBORDER:
@ -682,13 +683,11 @@ scteken_param(void *arg, int cmd, unsigned int value)
sc_set_border(scp, scp->border);
break;
case TP_SHOWCURSOR:
if (value) {
sc_change_cursor_shape(scp,
CONS_RESET_CURSOR|CONS_LOCAL_CURSOR, -1, -1);
} else {
sc_change_cursor_shape(scp,
CONS_HIDDEN_CURSOR|CONS_LOCAL_CURSOR, -1, -1);
}
if (value != 0)
flags = scp->curr_curs_attr.flags & ~CONS_HIDDEN_CURSOR;
else
flags = scp->curr_curs_attr.flags | CONS_HIDDEN_CURSOR;
sc_change_cursor_shape(scp, flags | CONS_LOCAL_CURSOR, -1, -1);
break;
case TP_SWITCHVT:
sc_switch_scr(scp->sc, value);