https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=225096
--- Comment #3 from Ed Schouten <[email protected]> --- 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. _______________________________________________ [email protected] mailing list https://lists.freebsd.org/mailman/listinfo/freebsd-bugs To unsubscribe, send any mail to "[email protected]"
