summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIngo Schwarze <schwarze@cvs.openbsd.org>2022-04-26 14:46:31 +0000
committerIngo Schwarze <schwarze@cvs.openbsd.org>2022-04-26 14:46:31 +0000
commit15a70fb6ca6f0d2fd59ce04393d65dcbd7d5e33c (patch)
tree20e3ab2b4ea13e73e3a2df0b0c31641e4c3f6ec1
parentb77668bae395acdd312e71014582db3b7a444698 (diff)
At the end of every tbl(7) cell, clear the \z state.
This is needed because the TERMP_MULTICOL mode is designed such that term_tbl() buffers all the cells of the table row before the normal reset logic near the end of term_flushln() can be reached. This fixes an assertion failure triggered by \z near the end of a table cell, found by tb@ using afl(1).
-rw-r--r--regress/usr.bin/mandoc/tbl/data/Makefile11
-rw-r--r--regress/usr.bin/mandoc/tbl/data/escz.in14
-rw-r--r--regress/usr.bin/mandoc/tbl/data/escz.out_ascii14
-rw-r--r--regress/usr.bin/mandoc/tbl/data/escz_end.in14
-rw-r--r--regress/usr.bin/mandoc/tbl/data/escz_end.out_ascii14
-rw-r--r--usr.bin/mandoc/tbl_term.c5
6 files changed, 66 insertions, 6 deletions
diff --git a/regress/usr.bin/mandoc/tbl/data/Makefile b/regress/usr.bin/mandoc/tbl/data/Makefile
index b1dcfa08114..6db686b9dd9 100644
--- a/regress/usr.bin/mandoc/tbl/data/Makefile
+++ b/regress/usr.bin/mandoc/tbl/data/Makefile
@@ -1,13 +1,16 @@
-# $OpenBSD: Makefile,v 1.5 2019/07/18 14:38:47 schwarze Exp $
+# $OpenBSD: Makefile,v 1.6 2022/04/26 14:46:30 schwarze Exp $
REGRESS_TARGETS = blankline block_empty block_unclosed block_width
-REGRESS_TARGETS += block_wrap empty insert
+REGRESS_TARGETS += block_wrap empty escz escz_end insert
LINT_TARGETS = block_unclosed empty insert
-# groff-1.22.3 defect:
+# groff-1.23.0 defects:
# - When a table ends in the middle of a block,
# GNU eqn produces no output whatsoever for the whole table.
+# - When a table cell ends with a no-advance escape sequence,
+# the next cell is appended to the current cell instead of
+# aligning it to the next column.
-SKIP_GROFF = block_unclosed
+SKIP_GROFF = block_unclosed escz_end
.include <bsd.regress.mk>
diff --git a/regress/usr.bin/mandoc/tbl/data/escz.in b/regress/usr.bin/mandoc/tbl/data/escz.in
new file mode 100644
index 00000000000..406df48fceb
--- /dev/null
+++ b/regress/usr.bin/mandoc/tbl/data/escz.in
@@ -0,0 +1,14 @@
+.\" $OpenBSD: escz.in,v 1.1 2022/04/26 14:46:30 schwarze Exp $
+.TH TBL-DATA-ESCZ 1 "April 26, 2022"
+.SH NAME
+tbl-data-escz \- non-advancing glyph at the end of a tbl(7) cell
+.SH DESCRIPTION
+initial text
+.TS
+tab(:);
+LL.
+a:b
+c\zd:e
+.TE
+.sp
+final text
diff --git a/regress/usr.bin/mandoc/tbl/data/escz.out_ascii b/regress/usr.bin/mandoc/tbl/data/escz.out_ascii
new file mode 100644
index 00000000000..486ab3d0cd0
--- /dev/null
+++ b/regress/usr.bin/mandoc/tbl/data/escz.out_ascii
@@ -0,0 +1,14 @@
+TBL-DATA-ESCZ(1) General Commands Manual TBL-DATA-ESCZ(1)
+
+NNAAMMEE
+ tbl-data-escz - non-advancing glyph at the end of a tbl(7) cell
+
+DDEESSCCRRIIPPTTIIOONN
+ initial text
+
+ a b
+ cd e
+
+ final text
+
+OpenBSD April 26, 2022 TBL-DATA-ESCZ(1)
diff --git a/regress/usr.bin/mandoc/tbl/data/escz_end.in b/regress/usr.bin/mandoc/tbl/data/escz_end.in
new file mode 100644
index 00000000000..5149f5d641e
--- /dev/null
+++ b/regress/usr.bin/mandoc/tbl/data/escz_end.in
@@ -0,0 +1,14 @@
+.\" $OpenBSD: escz_end.in,v 1.1 2022/04/26 14:46:30 schwarze Exp $
+.TH TBL-DATA-ESCZ_END 1 "April 26, 2022"
+.SH NAME
+tbl-data-escz_end \- z escape sequence at the end of a tbl(7) cell
+.SH DESCRIPTION
+initial text
+.TS
+tab(:);
+LLL.
+a:b:c
+d\z:e:f
+.TE
+.sp
+final text
diff --git a/regress/usr.bin/mandoc/tbl/data/escz_end.out_ascii b/regress/usr.bin/mandoc/tbl/data/escz_end.out_ascii
new file mode 100644
index 00000000000..f195c9c29b0
--- /dev/null
+++ b/regress/usr.bin/mandoc/tbl/data/escz_end.out_ascii
@@ -0,0 +1,14 @@
+TBL-DATA-ESCZ_END(1) General Commands Manual TBL-DATA-ESCZ_END(1)
+
+NNAAMMEE
+ tbl-data-escz_end - z escape sequence at the end of a tbl(7) cell
+
+DDEESSCCRRIIPPTTIIOONN
+ initial text
+
+ a b c
+ d e f
+
+ final text
+
+OpenBSD April 26, 2022 TBL-DATA-ESCZ_END(1)
diff --git a/usr.bin/mandoc/tbl_term.c b/usr.bin/mandoc/tbl_term.c
index c054fde2e42..f20aec9adb6 100644
--- a/usr.bin/mandoc/tbl_term.c
+++ b/usr.bin/mandoc/tbl_term.c
@@ -1,7 +1,7 @@
-/* $OpenBSD: tbl_term.c,v 1.64 2022/04/08 16:53:40 schwarze Exp $ */
+/* $OpenBSD: tbl_term.c,v 1.65 2022/04/26 14:46:30 schwarze Exp $ */
/*
+ * Copyright (c) 2011-2022 Ingo Schwarze <schwarze@openbsd.org>
* Copyright (c) 2009, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
- * Copyright (c) 2011-2021 Ingo Schwarze <schwarze@openbsd.org>
*
* Permission to use, copy, modify, and distribute this software for any
* purpose with or without fee is hereby granted, provided that the above
@@ -288,6 +288,7 @@ term_tbl(struct termp *tp, const struct tbl_span *sp)
}
tp->tcol++;
tp->col = 0;
+ tp->flags &= ~(TERMP_BACKAFTER | TERMP_BACKBEFORE);
tbl_data(tp, sp->opts, cp, dp, tp->tbl.cols + ic);
if (dp != NULL &&
(ic || sp->layout->first->pos != TBL_CELL_SPAN)) {