diff options
-rw-r--r-- | regress/roff/ti/Makefile | 10 | ||||
-rw-r--r-- | regress/roff/ti/wide.in | 19 | ||||
-rw-r--r-- | regress/roff/ti/wide.out_ascii | 22 | ||||
-rw-r--r-- | roff_term.c | 32 |
4 files changed, 70 insertions, 13 deletions
diff --git a/regress/roff/ti/Makefile b/regress/roff/ti/Makefile index 4ac4cde0..aa0b1f97 100644 --- a/regress/roff/ti/Makefile +++ b/regress/roff/ti/Makefile @@ -1,5 +1,11 @@ -# $OpenBSD: Makefile,v 1.1 2017/05/07 17:30:58 schwarze Exp $ +# $OpenBSD: Makefile,v 1.2 2020/09/03 17:37:06 schwarze Exp $ -REGRESS_TARGETS = basic-mdoc basic-man +REGRESS_TARGETS = basic-mdoc basic-man wide + +# groff-1.22.4 defect: +# - Excessive temporare indentations cause excessive line lengths +# instead of being truncated to resonable indentations. + +SKIP_GROFF = wide .include <bsd.regress.mk> diff --git a/regress/roff/ti/wide.in b/regress/roff/ti/wide.in new file mode 100644 index 00000000..46a1f90b --- /dev/null +++ b/regress/roff/ti/wide.in @@ -0,0 +1,19 @@ +.\" $OpenBSD: wide.in,v 1.1 2020/09/03 17:37:06 schwarze Exp $ +.TH TI-WIDE 1 "September 3, 2020" +.SH NAME +ti-wide \- excessive temporary indentation +.SH DESCRIPTION +Absolute .ti of more than 72n: +.ti 80n +max +.RS 40n +Indented block plus excessive relative indentation: +.ti +40n +max +.RE +.RS 66n +Block beyond 72n: +.ti +4n +same +.RE +Back to normal. diff --git a/regress/roff/ti/wide.out_ascii b/regress/roff/ti/wide.out_ascii new file mode 100644 index 00000000..95a98ddc --- /dev/null +++ b/regress/roff/ti/wide.out_ascii @@ -0,0 +1,22 @@ +TI-WIDE(1) General Commands Manual TI-WIDE(1) + + + +NNAAMMEE + ti-wide - excessive temporary indentation + +DDEESSCCRRIIPPTTIIOONN + Absolute .ti of more than 72n: + max + Indented block plus excessive + relative indentation: + max + Block + beyond + 72n: + same + Back to normal. + + + +OpenBSD September 3, 2020 TI-WIDE(1) diff --git a/roff_term.c b/roff_term.c index 72d1887e..ff71dc35 100644 --- a/roff_term.c +++ b/roff_term.c @@ -210,6 +210,7 @@ roff_term_pre_ti(ROFF_TERM_ARGS) { struct roffsu su; const char *cp; + const size_t maxoff = 72; int len, sign; roff_term_pre_br(p, n); @@ -230,17 +231,26 @@ roff_term_pre_ti(ROFF_TERM_ARGS) return; len = term_hen(p, &su); - if (sign == 0) { + switch (sign) { + case 1: + if (p->tcol->offset + len <= maxoff) + p->ti = len; + else if (p->tcol->offset < maxoff) + p->ti = maxoff - p->tcol->offset; + else + p->ti = 0; + break; + case -1: + if ((size_t)len < p->tcol->offset) + p->ti = -len; + else + p->ti = -p->tcol->offset; + break; + default: + if ((size_t)len > maxoff) + len = maxoff; p->ti = len - p->tcol->offset; - p->tcol->offset = len; - } else if (sign == 1) { - p->ti = len; - p->tcol->offset += len; - } else if ((size_t)len < p->tcol->offset) { - p->ti = -len; - p->tcol->offset -= len; - } else { - p->ti = -p->tcol->offset; - p->tcol->offset = 0; + break; } + p->tcol->offset += p->ti; } |