diff options
author | Ingo Schwarze <schwarze@openbsd.org> | 2022-08-15 13:04:43 +0000 |
---|---|---|
committer | Ingo Schwarze <schwarze@openbsd.org> | 2022-08-15 13:04:43 +0000 |
commit | ffa876e5886e09f94c3fe78fbef3cc7ebcd8a611 (patch) | |
tree | d6755f20efaea5d5571591b525c8bde74110b328 | |
parent | b8c2ddec80f36a42b137370c233bb4a858567168 (diff) | |
download | mandoc-ffa876e5886e09f94c3fe78fbef3cc7ebcd8a611.tar.gz |
In GNU, Heirloom, and Plan 9 roff, tab positions apply to *input* lines,
not to *output* lines. In particular, if an input line gets broken in
fill mode and a tab occurs in the second output line, it advances to a
position of at least (width of the first output line) + (width of a
space character even though this is never printed) + (width of the part
of the second output line that precedes the tab).
Implement the same logic in mandoc.
Again, do not use tabs in filled text: they have surprising effects,
including this one.
-rw-r--r-- | man_term.c | 1 | ||||
-rw-r--r-- | term.c | 17 | ||||
-rw-r--r-- | term.h | 1 |
3 files changed, 12 insertions, 7 deletions
@@ -975,6 +975,7 @@ out: ! (p->flags & (TERMP_NOBREAK | TERMP_NONEWLINE)) && (n->next == NULL || n->next->flags & NODE_LINE)) { p->flags |= TERMP_BRNEVER | TERMP_NOSPACE; + p->tcol->taboff = 0; if (n->string != NULL && *n->string != '\0') term_flushln(p); else @@ -157,6 +157,7 @@ term_flushln(struct termp *p) /* Finally, print the field content. */ term_field(p, vbl, nbr); + p->tcol->taboff += vbr + (*p->width)(p, ' '); /* * If there is no text left in the field, exit the loop. @@ -309,7 +310,9 @@ term_fill(struct termp *p, size_t *nbr, size_t *vbr, size_t vtarget) default: switch (p->tcol->buf[ic]) { case '\t': + vis += p->tcol->taboff; vis = term_tab_next(vis); + vis -= p->tcol->taboff; break; case ASCII_NBRSP: /* Non-breakable space. */ p->tcol->buf[ic] = ' '; @@ -348,8 +351,8 @@ term_field(struct termp *p, size_t vbl, size_t nbr) { size_t ic; /* Character position in the input buffer. */ size_t vis; /* Visual position of the current character. */ + size_t vt; /* Visual position including tab offset. */ size_t dv; /* Visual width of the current character. */ - size_t vn; /* Visual position of the next character. */ vis = 0; for (ic = p->tcol->col; ic < nbr; ic++) { @@ -364,13 +367,13 @@ term_field(struct termp *p, size_t vbl, size_t nbr) case ASCII_BREAK: continue; case '\t': - vn = term_tab_next(vis); - vbl += vn - vis; - vis = vn; - continue; case ' ': case ASCII_NBRSP: - dv = (*p->width)(p, ' '); + if (p->tcol->buf[ic] == '\t') { + vt = p->tcol->taboff + vis; + dv = term_tab_next(vt) - vt; + } else + dv = (*p->width)(p, ' '); vbl += dv; vis += dv; continue; @@ -432,7 +435,7 @@ endline(struct termp *p) void term_newln(struct termp *p) { - + p->tcol->taboff = 0; p->flags |= TERMP_NOSPACE; if (p->tcol->lastcol || p->viscol) term_flushln(p); @@ -56,6 +56,7 @@ struct termp_col { size_t col; /* Byte in buf to be written. */ size_t rmargin; /* Current right margin. */ size_t offset; /* Current left margin. */ + size_t taboff; /* Offset for literal tabs. */ }; struct termp { |