From 42d5f8ddfdcc19abca690eba6b3c1bc230e43079 Mon Sep 17 00:00:00 2001
From: Ingo Schwarze <schwarze@openbsd.org>
Date: Fri, 13 Mar 2020 15:32:28 +0000
Subject: Split tagging into a validation part including prioritization in
 tag.{h,c} and {mdoc,man}_validate.c and into a formatting part including
 command line argument checking in term_tag.{h,c}, html.c, and
 {mdoc|man}_{term|html}.c.

Immediate functional benefits include:
* Improved prioritization of automatic tags for .Em and .Sy.
* Avoiding bogus automatic tags when .Em, .Fn, or .Sy are explicitly tagged.
* Explicit tagging of .Er and .Fl now works in HTML output.
* Automatic tagging of .IP and .TP now works in HTML output.
But mainly, this patch provides clean earth to build further improvements on.

Technical changes:
* Main program: Write a tag file for ASCII and UTF-8 output only.
* All formatters: There is no more need to delay writing the tags.
* mdoc(7)+man(7) formatters: No more need for elaborate syntax tree inspection.
* HTML formatter: If available, use the "string" attribute as the tag.
* HTML formatter: New function to write permalinks, to reduce code duplication.

Style cleanup in the vicinity while here:
* mdoc(7) terminal formatter: To set up bold font for children,
defer to termp_bold_pre() rather than calling term_fontpush() manually.
* mdoc(7) terminal formatter: Garbage collect some duplicate functions.
* mdoc(7) HTML formatter: Unify <code> handling, delete redundant functions.
* Where possible, use switch statements rather than if cascades.
* Get rid of some more Yoda notation.

The necessity for such changes was first discussed with kn@, but i didn't
bother him with a request to review the resulting -673/+782 line patch.
---
 regress/mdoc/Em/Makefile         |  6 ++++--
 regress/mdoc/Em/tag.in           | 23 +++++++++++++++++++++++
 regress/mdoc/Em/tag.out_ascii    | 17 +++++++++++++++++
 regress/mdoc/Em/tag.out_html     | 10 ++++++++++
 regress/mdoc/Em/tag.out_markdown | 25 +++++++++++++++++++++++++
 regress/mdoc/Em/tag.out_tag      |  5 +++++
 6 files changed, 84 insertions(+), 2 deletions(-)
 create mode 100644 regress/mdoc/Em/tag.in
 create mode 100644 regress/mdoc/Em/tag.out_ascii
 create mode 100644 regress/mdoc/Em/tag.out_html
 create mode 100644 regress/mdoc/Em/tag.out_markdown
 create mode 100644 regress/mdoc/Em/tag.out_tag

(limited to 'regress/mdoc/Em')

diff --git a/regress/mdoc/Em/Makefile b/regress/mdoc/Em/Makefile
index 13de4337..2b500444 100644
--- a/regress/mdoc/Em/Makefile
+++ b/regress/mdoc/Em/Makefile
@@ -1,6 +1,8 @@
-# $OpenBSD: Makefile,v 1.3 2014/11/17 06:44:35 schwarze Exp $
+# $OpenBSD: Makefile,v 1.6 2020/03/13 00:31:05 schwarze Exp $
 
-REGRESS_TARGETS	= font noarg punct
+REGRESS_TARGETS	= font noarg punct tag
+TAG_TARGETS	= tag
 LINT_TARGETS	= noarg punct
+HTML_TARGETS	= tag
 
 .include <bsd.regress.mk>
diff --git a/regress/mdoc/Em/tag.in b/regress/mdoc/Em/tag.in
new file mode 100644
index 00000000..9d309835
--- /dev/null
+++ b/regress/mdoc/Em/tag.in
@@ -0,0 +1,23 @@
+.\" $OpenBSD: tag.in,v 1.1 2020/03/13 00:31:05 schwarze Exp $
+.Dd $Mdocdate$
+.Dt EM-TAG 1
+.Os
+.Sh NAME
+.Nm Em-tag
+.Nd tagging of emphasis macros
+.Sh DESCRIPTION
+BEGINTEST
+.Bl -tag -width Ds
+.It Em one | two
+text
+.It Xo
+.Em three
+.Xc
+text
+.El
+.Em four
+.Em one
+.Tg explicit
+.Em five
+.Pp
+ENDTEST
diff --git a/regress/mdoc/Em/tag.out_ascii b/regress/mdoc/Em/tag.out_ascii
new file mode 100644
index 00000000..74f8212b
--- /dev/null
+++ b/regress/mdoc/Em/tag.out_ascii
@@ -0,0 +1,17 @@
+EM-TAG(1)                   General Commands Manual                  EM-TAG(1)
+
+NNAAMMEE
+     EEmm--ttaagg - tagging of emphasis macros
+
+DDEESSCCRRIIPPTTIIOONN
+     BEGINTEST
+
+     _o_n_e | _t_w_o
+             text
+
+     _t_h_r_e_e   text
+     _f_o_u_r _o_n_e _f_i_v_e
+
+     ENDTEST
+
+OpenBSD                         March 13, 2020                         OpenBSD
diff --git a/regress/mdoc/Em/tag.out_html b/regress/mdoc/Em/tag.out_html
new file mode 100644
index 00000000..4046673f
--- /dev/null
+++ b/regress/mdoc/Em/tag.out_html
@@ -0,0 +1,10 @@
+<dl class="Bl-tag">
+  <dt><a class="permalink" href="#one"><i class="Em" id="one">one</i></a> |
+    <a class="permalink" href="#two"><i class="Em" id="two">two</i></a></dt>
+  <dd>text</dd>
+  <dt><a class="permalink" href="#three"><i class="Em" id="three">three</i></a></dt>
+  <dd>text</dd>
+</dl>
+<a class="permalink" href="#four"><i class="Em" id="four">four</i></a>
+  <i class="Em">one</i>
+  <a class="permalink" href="#explicit"><i class="Em" id="explicit">five</i></a>
diff --git a/regress/mdoc/Em/tag.out_markdown b/regress/mdoc/Em/tag.out_markdown
new file mode 100644
index 00000000..4c107159
--- /dev/null
+++ b/regress/mdoc/Em/tag.out_markdown
@@ -0,0 +1,25 @@
+EM-TAG(1) - General Commands Manual
+
+# NAME
+
+**Em-tag** - tagging of emphasis macros
+
+# DESCRIPTION
+
+BEGINTEST
+
+*one* | *two*
+
+> text
+
+*three*
+
+> text
+
+*four*
+*one*
+*five*
+
+ENDTEST
+
+OpenBSD - March 13, 2020
diff --git a/regress/mdoc/Em/tag.out_tag b/regress/mdoc/Em/tag.out_tag
new file mode 100644
index 00000000..c2fbaf59
--- /dev/null
+++ b/regress/mdoc/Em/tag.out_tag
@@ -0,0 +1,5 @@
+one 9
+two 9
+three 12
+four 13
+explicit 13
-- 
cgit