Re: tweek rcctl.8 spacing

2016-10-10 Thread Ingo Schwarze
Hi,

Jan Stary wrote on Mon, Oct 10, 2016 at 02:22:57PM +0200:

> The postscript output of 'man -Tps rcctl' makes the 'stopped'
> list item of 'rcctl ls' break a line, while all the other items
> are compact one-liners. The diff below changes the width,
> which is the same for ascci, but "stopped" is slightly wider
> than "started" in the postscript output.

Committed, thanks.
  Ingo


> Index: rcctl.8
> ===
> RCS file: /cvs/src/usr.sbin/rcctl/rcctl.8,v
> retrieving revision 1.33
> diff -u -p -r1.33 rcctl.8
> --- rcctl.8   19 Jun 2016 10:54:20 -  1.33
> +++ rcctl.8   10 Oct 2016 12:16:59 -
> @@ -108,7 +108,7 @@ Display a list of services and daemons m
>  .Ar lsarg ,
>  which can be one of:
>  .Pp
> -.Bl -tag -width started -offset indent -compact
> +.Bl -tag -width stopped -offset indent -compact
>  .It Cm all
>  all services and daemons
>  .It Cm failed
> 



Re: tweek rcctl.8 spacing

2016-10-10 Thread Jan Stary
> > The postscript output of 'man -Tps rcctl' makes the 'stopped'
> > list item of 'rcctl ls' break a line, while all the other items
> > are compact one-liners. The diff below changes the width,
> > which is the same for ascci, but "stopped" is slightly wider
> > than "started" in the postscript output.
> 
> In practice, we cannot expect manual writers - who are mortal
> developers and not typography gods - to pay attention to proportional
> font glyph widths and kerning.  Those who are - like Doug McIlroy
> or Brian Kernighan - would do the right thing instinctively anyway,
> but the others just shouldn't need to bother.

(Needless to say, this was no typograhical insight on my part,
I simply printed out the postscript for once instead of plain man.)



Re: tweek rcctl.8 spacing

2016-10-10 Thread Ingo Schwarze
Hi,

Jan Stary wrote on Mon, Oct 10, 2016 at 02:22:57PM +0200:

> The postscript output of 'man -Tps rcctl' makes the 'stopped'
> list item of 'rcctl ls' break a line, while all the other items
> are compact one-liners. The diff below changes the width,
> which is the same for ascci, but "stopped" is slightly wider
> than "started" in the postscript output.

Gaah.  Strictly speaking, Jan is right.

In practice, we cannot expect manual writers - who are mortal
developers and not typography gods - to pay attention to proportional
font glyph widths and kerning.  Those who are - like Doug McIlroy
or Brian Kernighan - would do the right thing instinctively anyway,
but the others just shouldn't need to bother.

Adding this as yet another item to Jason's "tweak previous"
checklist sounds like a bad idea, too - unless i can support it
by adding a warning to -Tlint, but that would cause some work
and seems low priority at best.

Jason, do you agree to simply ignore the problem in general, but
to commit such patches whenever people care and send specific,
correct patches?

In any case, this one is OK schwarze@.

Yours,
  Ingo


> Index: rcctl.8
> ===
> RCS file: /cvs/src/usr.sbin/rcctl/rcctl.8,v
> retrieving revision 1.33
> diff -u -p -r1.33 rcctl.8
> --- rcctl.8   19 Jun 2016 10:54:20 -  1.33
> +++ rcctl.8   10 Oct 2016 12:16:59 -
> @@ -108,7 +108,7 @@ Display a list of services and daemons m
>  .Ar lsarg ,
>  which can be one of:
>  .Pp
> -.Bl -tag -width started -offset indent -compact
> +.Bl -tag -width stopped -offset indent -compact
>  .It Cm all
>  all services and daemons
>  .It Cm failed



tweek rcctl.8 spacing

2016-10-10 Thread Jan Stary
The postscript output of 'man -Tps rcctl' makes the 'stopped'
list item of 'rcctl ls' break a line, while all the other items
are compact one-liners. The diff below changes the width,
which is the same for ascci, but "stopped" is slightly wider
than "started" in the postscript output.

Jan


Index: rcctl.8
===
RCS file: /cvs/src/usr.sbin/rcctl/rcctl.8,v
retrieving revision 1.33
diff -u -p -r1.33 rcctl.8
--- rcctl.8 19 Jun 2016 10:54:20 -  1.33
+++ rcctl.8 10 Oct 2016 12:16:59 -
@@ -108,7 +108,7 @@ Display a list of services and daemons m
 .Ar lsarg ,
 which can be one of:
 .Pp
-.Bl -tag -width started -offset indent -compact
+.Bl -tag -width stopped -offset indent -compact
 .It Cm all
 all services and daemons
 .It Cm failed