The mdoc(7) \*(Ba predefined string actually forces roman font;

that's stupid because it may break enclosing font changes,
but let's do the same for groff bug compatibility.

--> Never use \*(Ba, use just plain "|"! <--

Also, predefined strings are already expanded by the roff(7) parser,
so the mdoc(7) parser has to look for the expanded string.

OpenBSD rev. mdoc.c 1.90 and predefs.in 1.3

3 files changed, 2 insertions, 14 deletions

diff --git a/TODO b/TODO
index 89525db0..407823f5 100644
--- a/TODO
+++ b/TODO
@@ -203,11 +203,6 @@
   is just "o\bo".
   see for example OpenBSD ksh(1)
 
-- The characters "|" and "\*(Ba" should never be bold,
-  not even in the middle of a word, e.g. ".Cm b\*(Bac" in
-  "mknod [-m mode] name b|c major minor"
-  in OpenBSD ksh(1)
-
 - A bogus .Pp between two .It must not produce a double blank line,
   see between -R and -r in OpenBSD rm(1), before "update" in mount(8),
   or in DIAGNOSTICS in init(8), or before "is always true" in ksh(1).
@@ -270,13 +265,6 @@
   found while talking to Chris Bennett
 
 - In man(7), the sequence
-    .HP
-    one line of regular text
-    .SH
-  should not produce two blank lines before the .SH,
-  see for example named-checkconf(8).
-
-- In man(7), the sequence
     .SH HEADER
     <blank line>
     .PP
diff --git a/mdoc.c b/mdoc.c
index 90cc8169..e5486a52 100644
--- a/mdoc.c
+++ b/mdoc.c
@@ -982,7 +982,7 @@ mdoc_isdelim(const char *p)
 
 	if (0 == strcmp(p + 1, "."))
 		return(DELIM_CLOSE);
-	if (0 == strcmp(p + 1, "*(Ba"))
+	if (0 == strcmp(p + 1, "fR|\\fP"))
 		return(DELIM_MIDDLE);
 
 	return(DELIM_NONE);
diff --git a/predefs.in b/predefs.in
index decf661e..377f8309 100644
--- a/predefs.in
+++ b/predefs.in
@@ -26,7 +26,7 @@
  */
 
 PREDEF("Am", "&")
-PREDEF("Ba", "|")
+PREDEF("Ba", "\\fR|\\fP")
 PREDEF("Ge", "\\(>=")
 PREDEF("Gt", ">")
 PREDEF("If", "infinity")