[Bug 225096] [libteken] [PATCH] Add reverse wrap around to libteken
bugzilla-noreply at freebsd.org
bugzilla-noreply at freebsd.org
Tue May 21 21:01:43 UTC 2019
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=225096
--- Comment #3 from Ed Schouten <ed at FreeBSD.org> ---
Hey! The patch looks decent, but I'm wondering whether it's possible to merge
some of the now redundant logic in teken_subr_backspace(), e.g.:
...
} else {
if (t->t_cursor.tp_col == 0) {
if ((t->t_stateflags & TS_REVWRAP) == 0)
return;
t->t_cursor.tp_row--;
t->t_cursor.tp_col = t->t_winsize.tp_col - 2;
} else {
t->t_cursor.tp_col--;
t->t_stateflags &= ~TS_WRAPPED;
}
}
Also, I think you might need to take TS_WRAPPED into account. Depending on the
value of that flag, you may need to jump to 't->t_winsize.tp_col - 1' or
't->t_winsize.tp_col - 2'.
Also, feel free to remove the teken_printf() calls there. They were just a
debugging aid to display unsupported escape sequences.
--
You are receiving this mail because:
You are the assignee for the bug.
More information about the freebsd-bugs
mailing list