From a20be95dec7797c55257792a32b6b39bdbdfe595 Mon Sep 17 00:00:00 2001
From: Ingo Schwarze <schwarze@openbsd.org>
Date: Tue, 30 May 2017 16:22:03 +0000
Subject: Macro argument quoting does not prevent recognition of punctuation
 and of called macros.

This bug affects almost all macros, and fixing it simplifies the
code.  It is amazing that the bogus ARGS_QWORD feature got implemented
in the first place, and then carrier along for more than eight years
without anybody ever noticing that it was pointless.

Reported by Leah Neukirchen <leah at vuxu dot org>, found on Void Linux.
---
 regress/mdoc/Li/punct.in           | 10 +++++-----
 regress/mdoc/Li/punct.out_ascii    |  2 +-
 regress/mdoc/Li/punct.out_lint     |  1 +
 regress/mdoc/Li/punct.out_markdown |  4 ++--
 4 files changed, 9 insertions(+), 8 deletions(-)

(limited to 'regress/mdoc/Li')

diff --git a/regress/mdoc/Li/punct.in b/regress/mdoc/Li/punct.in
index 331e55da..02565770 100644
--- a/regress/mdoc/Li/punct.in
+++ b/regress/mdoc/Li/punct.in
@@ -7,7 +7,7 @@
 .Sh DESCRIPTION
 Leading punctuation:
 .Li ( b
-.Li [ b
+.Li "[" b
 .Li | b
 .Li . b
 .Li , b
@@ -23,7 +23,7 @@ Trailing punctuation:
 .Li a [
 .Li a |
 .Li a .
-.Li a ,
+.Li a ","
 .Li a ;
 .Li a :
 .Li a ?
@@ -37,11 +37,11 @@ Middle punctuation:
 .Li a | b
 .Li a . b
 .Li a , b
-.Li a ; b
+.Li a ";" b
 .Li a : b
 .Li a ? b
 .Li a ! b
-.Li a ) b
+.Li a ")" b
 .Li a ] b
 .Pp
 Isolated punctuation:
@@ -78,6 +78,6 @@ Multiple punctuation:
 .Li [ ( arg ) ] .
 .Pp
 Quoted:
-.Li "a . b Nm"
+.Li "a . b Nm" "Sy" bold
 .Li ". b Nm"
 .Li "."
diff --git a/regress/mdoc/Li/punct.out_ascii b/regress/mdoc/Li/punct.out_ascii
index ae7d3f62..d0de6c9a 100644
--- a/regress/mdoc/Li/punct.out_ascii
+++ b/regress/mdoc/Li/punct.out_ascii
@@ -21,6 +21,6 @@ DDEESSCCRRIIPPTTIIOONN
 
      Multiple punctuation: [(arg)].
 
-     Quoted: a . b Nm . b Nm .
+     Quoted: a . b Nm bboolldd . b Nm .
 
 OpenBSD                        November 17, 2014                       OpenBSD
diff --git a/regress/mdoc/Li/punct.out_lint b/regress/mdoc/Li/punct.out_lint
index 3abdf12f..bdce45c6 100644
--- a/regress/mdoc/Li/punct.out_lint
+++ b/regress/mdoc/Li/punct.out_lint
@@ -22,3 +22,4 @@ mandoc: punct.in:70:7: WARNING: skipping empty macro: Li
 mandoc: punct.in:71:7: WARNING: skipping empty macro: Li
 mandoc: punct.in:74:7: WARNING: skipping empty macro: Li
 mandoc: punct.in:75:7: WARNING: skipping empty macro: Li
+mandoc: punct.in:83:2: WARNING: skipping empty macro: Li
diff --git a/regress/mdoc/Li/punct.out_markdown b/regress/mdoc/Li/punct.out_markdown
index fcfc83b3..97110e22 100644
--- a/regress/mdoc/Li/punct.out_markdown
+++ b/regress/mdoc/Li/punct.out_markdown
@@ -79,8 +79,8 @@ Multiple punctuation:
 \[(`arg`)].
 
 Quoted:
-`a . b Nm`
+`a . b Nm` **bold**
 `. b Nm`
-`.`
+.
 
 OpenBSD - November 17, 2014
-- 
cgit v1.2.3