Hi Joerg,

Joerg Jung wrote on Mon, Nov 09, 2015 at 10:31:10PM +0100:

> Index: patches/patch-dwm_1
> ===================================================================
> RCS file: patches/patch-dwm_1
> diff -N patches/patch-dwm_1
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ patches/patch-dwm_1       9 Nov 2015 21:30:56 -0000
> @@ -0,0 +1,21 @@
> +$OpenBSD$
> +--- dwm.1.orig       Mon Nov  9 22:25:40 2015
> ++++ dwm.1    Mon Nov  9 22:26:32 2015
> +@@ -57,7 +57,7 @@ click on a tag label adds/removes that tag to/from the
> + .TP
> + .B Mod1\-Shift\-Return
> + Start
> +-.BR st(1).
> ++.BR uxterm(1).

The line

  .BR uxterm(1).

makes no sense.  You don't need a font alternation macro if you
just have one single argument.

>From .Xr, mandoc -Tman produces just "uxterm(1)" without any
markup.  The man(7) manual recommends ".BR uxterm ( 1 ).",
i.e. keeping the punctuation out of bold face.  But for
consistency with the rest of the page, you probably want

  .BR uxterm (1).

Oh well, there is no consistent style for man(7) manuals, anyway.

This is not a comment on the port update - the words "man page patch"
just triggered my curiosity.  :-)

Yours,
  Ingo


> + .TP
> + .B Mod1\-,
> + Focus previous screen, if any.
> +@@ -151,7 +151,7 @@ dwm is customized by creating a custom config.h and (r
> + code. This keeps it fast, secure and simple.
> + .SH SEE ALSO
> + .BR dmenu (1),
> +-.BR st (1)
> ++.BR uxterm (1)
> + .SH BUGS
> + Java applications which use the XToolkit/XAWT backend may draw grey windows
> + only. The XToolkit/XAWT backend breaks ICCCM-compliance in recent JDK 1.5 
> and early
> 

Reply via email to