Re: [kbd] [PATCH] kbd/docs/man/man8misc: Fix some extra space and formatting

[Alexey Gladkov <gladkov.alexey@gmail.com>]

On Tue, Mar 24, 2020 at 12:43:48AM +0000, Bjarni Ingi Gislason wrote:

All your patches are applied. Thanks!

> Output is from: test-groff -b -e -mandoc -T utf8 -rF0 -t -w w -z
> 
>   [ "test-groff" is a developmental version of "groff" ]
> 
> Input file is ./docs/man/man8misc/clrunimap.8
> 
> troff: <docs/man/man8misc/clrunimap.8>:20: warning: trailing space
> ####
> 
> Input file is ./docs/man/man8misc/getunimap.8
> 
> troff: <docs/man/man8misc/getunimap.8>:17: warning: trailing space
> troff: <docs/man/man8misc/getunimap.8>:19: warning: trailing space
> troff: <docs/man/man8misc/getunimap.8>:41: warning: trailing space
> troff: <docs/man/man8misc/getunimap.8>:44: warning: trailing space
> troff: <docs/man/man8misc/getunimap.8>:59: warning: trailing space
> ####
> 
> Input file is ./docs/man/man8misc/mk_modmap.8
> 
> troff: <docs/man/man8misc/mk_modmap.8>:9: warning: trailing space
> troff: <docs/man/man8misc/mk_modmap.8>:21: warning: trailing space
> <docs/man/man8misc/mk_modmap.8>:34 (macro BR): only 1 argument, but more are expected
> ####
> 
> Input file is ./docs/man/man8misc/setlogcons.8
> 
> troff: <docs/man/man8misc/setlogcons.8>:4: warning: trailing space
> <docs/man/man8misc/setlogcons.8>:6 (macro BI): only 1 argument, but more are expected
> <docs/man/man8misc/setlogcons.8>:7 (macro BI): only 1 argument, but more are expected
> troff: <docs/man/man8misc/setlogcons.8>:17: warning: trailing space
> <docs/man/man8misc/setlogcons.8>:21 (macro BI): only 1 argument, but more are expected
> ####
> 
> Input file is ./docs/man/man8misc/setvesablank.8
> 
> troff: <docs/man/man8misc/setvesablank.8>:9: warning: trailing space
> troff: <docs/man/man8misc/setvesablank.8>:13: warning: trailing space
> ####
> 
> Input file is ./docs/man/man8misc/vcstime.8
> 
> <docs/man/man8misc/vcstime.8>:9 (macro BI): only 1 argument, but more are expected
> troff: <docs/man/man8misc/vcstime.8>:18: warning: trailing space
> <docs/man/man8misc/vcstime.8>:23 (macro BR): only 1 argument, but more are expected
> 
> Signed-off-by: Bjarni Ingi Gislason <bjarniig@rhi.hi.is>
> ####
> ---
>  docs/man/man8misc/clrunimap.8    |  2 +-
>  docs/man/man8misc/getunimap.8    | 10 +++++-----
>  docs/man/man8misc/mk_modmap.8    |  8 ++++----
>  docs/man/man8misc/setlogcons.8   | 10 +++++-----
>  docs/man/man8misc/setvesablank.8 |  8 ++++----
>  docs/man/man8misc/vcstime.8      |  6 +++---
>  6 files changed, 22 insertions(+), 22 deletions(-)
> 
> diff --git a/docs/man/man8misc/clrunimap.8 b/docs/man/man8misc/clrunimap.8
> index ca18f0a..1685e3e 100644
> --- a/docs/man/man8misc/clrunimap.8
> +++ b/docs/man/man8misc/clrunimap.8
> @@ -17,7 +17,7 @@ If you are not the maintainer of the kbd package, it is very unlikely
>  that you want to use this command. On many kernels the result will be
>  that your screen is black. On some kernels a reboot will be necessary.
>  .br
> -Use 
> +Use
>  .B "setfont -u"
>  to load a Unicode map for a console.
>  .SH SEE ALSO
> diff --git a/docs/man/man8misc/getunimap.8 b/docs/man/man8misc/getunimap.8
> index 087d85c..acf0a35 100644
> --- a/docs/man/man8misc/getunimap.8
> +++ b/docs/man/man8misc/getunimap.8
> @@ -14,9 +14,9 @@ The
>  program is old and obsolete. It is now part of
>  .B setfont (1).
>  .LP
> -The 
> +The
>  .B getunimap
> -program outputs the unicode map (also called a "Screen Font Map") 
> +program outputs the unicode map (also called a "Screen Font Map")
>  for the current console to standard output.
>  .LP
>  The
> @@ -38,10 +38,10 @@ Many unicode characters may be mapped to the same glyph.
>  .br
>  the Hash symbol
>  .B #
> -is used as a comment delimiter; characters after a hash sign (to the end of 
> +is used as a comment delimiter; characters after a hash sign (to the end of
>  the line) are comments.
>  .P
> -The 
> +The
>  .B \-s
>  option will sort and merge elements, sorting on font character.
>  Hence, it will produce output of the form:
> @@ -56,7 +56,7 @@ Hence, it will produce output of the form:
>  .P
>  The output of
>  .B getunimap
> -is of the form accepted by 
> +is of the form accepted by
>  .B setfont
>  and
>  .B psfaddtable
> diff --git a/docs/man/man8misc/mk_modmap.8 b/docs/man/man8misc/mk_modmap.8
> index 9db007f..ad7f7da 100644
> --- a/docs/man/man8misc/mk_modmap.8
> +++ b/docs/man/man8misc/mk_modmap.8
> @@ -6,7 +6,7 @@ mk_modmap \- translate a Linux keytable file into an xmodmap file
>  
>  .SH SYNOPSIS
>  .B mk_modmap
> -[ 
> +[
>  .B -v
>  ]
>  keymap_file
> @@ -18,7 +18,7 @@ command tries to translate a Linux console keytable file into
>  a file that can be parsed by xmodmap, and used within X.
>  It outputs the results to standard out.
>  .P
> -If the 
> +If the
>  .B -v
>  flag is used, verbose output is given.
>  .P
> @@ -31,7 +31,7 @@ keymap, so xmodmap will often be superflous.
>  .SH SEE ALSO
>  .BR dumpkeys (1),
>  .BR keymaps (5),
> -.BR xmodmap(1)
> +.BR xmodmap (1)
>  
>  
>  .\" .SH AUTHORS
> @@ -39,4 +39,4 @@ keymap, so xmodmap will often be superflous.
>  .\" .br
>  .\" Pablo Saratxaga <pablo@mandrakesoft.com>
>  .\" .br
> -.\" Manpage by Alastair McKinstry, <mckinstry@computer.org>
> \ No newline at end of file
> +.\" Manpage by Alastair McKinstry, <mckinstry@computer.org>
> diff --git a/docs/man/man8misc/setlogcons.8 b/docs/man/man8misc/setlogcons.8
> index 3b9870f..92dc67b 100644
> --- a/docs/man/man8misc/setlogcons.8
> +++ b/docs/man/man8misc/setlogcons.8
> @@ -1,10 +1,10 @@
>  .TH SETLOGCONS 8 "18 Apr 2004" "Console tools" "Linux User's Manual"
>  
>  .SH NAME
> -setlogcons \- Send kernel messages to console N 
> +setlogcons \- Send kernel messages to console N
>  .SH SYNOPSIS
> -.BI setlogcons
> -.BI N
> +.B setlogcons
> +.I N
>  
>  .SH DESCRIPTION
>  The command
> @@ -14,11 +14,11 @@ sets all kernel messges to the console specified as N.
>  .P
>  By default kernel messages are sent to the current console.
>  .P
> -To change the level of messages sent, use 
> +To change the level of messages sent, use
>  .B dmesg
>  
>  .SH "AUTHORS"
> -.BI setlogcons 
> +.B setlogcons
>  was originally written by Andries Brouwer for the kbd package.
>  .P
>  This manual page was written by Alastair McKinstry.
> diff --git a/docs/man/man8misc/setvesablank.8 b/docs/man/man8misc/setvesablank.8
> index 8a48867..b194eb2 100644
> --- a/docs/man/man8misc/setvesablank.8
> +++ b/docs/man/man8misc/setvesablank.8
> @@ -5,12 +5,12 @@ setvesablank \- Turn VESA screen blanking on or off
>  .SH SYNOPSIS
>  .I setvesablank
>  [ [
> -.I -b --blanking 
> -] 
> -.I on | ON | off | OFF 
> +.I -b --blanking
> +]
> +.I on | ON | off | OFF
>  ] [
>  .I -h --help
> -] 
> +]
>  [
>  .I -v --verbose
>  ]
> diff --git a/docs/man/man8misc/vcstime.8 b/docs/man/man8misc/vcstime.8
> index 419b762..b0869c5 100644
> --- a/docs/man/man8misc/vcstime.8
> +++ b/docs/man/man8misc/vcstime.8
> @@ -6,7 +6,7 @@
>  vcstime \- Show time in upper right hand corner of the console screen
>  
>  .SH SYNOPSIS
> -.BI vcstime
> +.B vcstime
>  
>  .SH DESCRIPTION
>  .PP
> @@ -15,11 +15,11 @@ shows the current time in the upper right\-hand corner of the console screen.
>  .PP
>  This simple program shows the current time in the corner of the console screen.
>  .PP
> -It needs to be run by root, in order to have write permissions to 
> +It needs to be run by root, in order to have write permissions to
>  .I /dev/vcsa
>  
>  
>  .SH "AUTHORS"
> -.BR vcstime
> +.B vcstime
>  was written by Andries Brouwer, based on  a suggestion by Miguel de Icaza.
>  This manual page was Written by Alastair McKinstry, Debian, Jan 2003.
> -- 
> 2.25.1
> _______________________________________________
> kbd mailing list
> kbd@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/kbd
> 

-- 
Rgrds, legion