groff-commit
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[groff] 03/04: man pages: Fix markup bugs exposed by man's .itc.


From: G. Branden Robinson
Subject: [groff] 03/04: man pages: Fix markup bugs exposed by man's .itc.
Date: Sun, 19 Nov 2017 21:38:38 -0500 (EST)

gbranden pushed a commit to branch master
in repository groff.

commit 0ec18f5125d498e9a0e6a1ec2cd52f61a882d002
Author: G. Branden Robinson <address@hidden>
Date:   Sun Nov 19 18:39:35 2017 -0500

    man pages: Fix markup bugs exposed by man's .itc.
    
    The following somewhat contentious change exposed (or caused, depending
    on your perspective) rendering problems in 2 (and only 2) of groff's man
    pages.
    
        commit 3549fd9f7bb9134e84a51e9bf82521b6af87a011
        Author: Bjarni Ingi Gislason <address@hidden>
        Date:   Fri Apr 28 21:27:09 2017 +0200
    
            Macro "itc" is needed to make escape "\c" useful
    
            Fixes https://savannah.gnu.org/bugs/?42907
    
    * contrib/hdtbl/groff_hdtbl.7.man: Page-local macros which changed the
      font family to Courier (constant-width) interacted poorly with the
      change.  A line continuation after calling the page-local .CI macro
      (Courier italic) did not "shut off" the font on the next line after
      the commit above was made.  Force a change back to the Times family
      by calling man(7)'s .RI with a dummy second argument.
    
    * man/groff_out.5.man: Replace
          .B D\~\c
          command
      with ".BR D \~command", which is more idiomatic anyway, and
      furthermore avoids the \c escape sequence altogether.
    
    Signed-off-by: G. Branden Robinson <address@hidden>
---
 contrib/hdtbl/groff_hdtbl.7.man | 27 +++++++++++++++++----------
 man/groff_out.5.man             | 10 ++++------
 2 files changed, 21 insertions(+), 16 deletions(-)

diff --git a/contrib/hdtbl/groff_hdtbl.7.man b/contrib/hdtbl/groff_hdtbl.7.man
index 0a648a6..129dd95 100644
--- a/contrib/hdtbl/groff_hdtbl.7.man
+++ b/contrib/hdtbl/groff_hdtbl.7.man
@@ -649,9 +649,10 @@ Horizontal alignment of the table, if it is smaller than 
the line width.
 Widths of table cells.
 .
 .CI w1\c
-,
+.RI , ""
 .CI w2\c
-, \[ell] are either numbers of type\~\c
+.RI , ""
+\[ell] are either numbers of type\~\c
 .CI n
 or natural numbers with the pseudo-scaling indicator
 .CR \[oq]%\[cq]\c
@@ -681,7 +682,8 @@ Height of the table.
 .
 If the table with its contents is lower than\~\c
 .CI n\c
-, the last row is stretched to this value.
+.RI , ""
+the last row is stretched to this value.
 .RE
 .RE
 .
@@ -734,10 +736,12 @@ Begin a new table row.
 The height of the row.
 .
 If a cell in the row is higher than\~\c
-.CI n
-this value is ignored; otherwise the row height is stretched to\~\c
 .CI n\c
-\&.
+.RI , ""
+this value is ignored;
+otherwise the row height is stretched to\~\c
+.CI n\c
+.RI . ""
 .RE
 .RE
 .
@@ -935,8 +939,9 @@ This argument is ignored for inner (nested) tables.
 .TP
 .CBI ".t*free " \[lB]n\[rB]
 Free the next held table or
-.CI n\c
-\~held tables.
+.CI n\~\c
+.RI held ""
+tables.
 .
 Call this utility macro to print tables which are held by using the
 .CR \[oq]hold\[cq]
@@ -1043,9 +1048,11 @@ The font style in use right before the table (string
 .XAA "fsz=\[aq]" "d1 \[lB]d2\[rB]" \[aq]
 A decimal or fractional factor
 .CI d1\c
-, by which the point size for the table is changed, and
+.RI , ""
+by which the point size for the table is changed, and
 .CI d2\c
-, by which the vertical line spacing is changed.
+.RI , ""
+by which the vertical line spacing is changed.
 .
 If
 .CI d2
diff --git a/man/groff_out.5.man b/man/groff_out.5.man
index 13a5bc2..a82794d 100644
--- a/man/groff_out.5.man
+++ b/man/groff_out.5.man
@@ -837,12 +837,10 @@ is followed by a fixed or variable number of integer 
arguments that
 are separated by a single space character.
 .
 A
-.B D\c
-\~command
+.BR D \~command
 may not be followed by another command on the same line (apart from a
 comment), so each
-.B D\c
-\~command
+.BR D \~command
 is terminated by a
 .I syntactical line
 .IR break .
@@ -890,8 +888,8 @@ escape sequence; see
 .
 .P
 Unknown
-.B D\c
-\~commands are assumed to be device-specific.
+.BR D \~commands
+are assumed to be device-specific.
 .
 Its arguments are parsed as strings; the whole information is then
 sent to the postprocessor.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]