util-linux.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Bjarni Ingi Gislason <bjarniig@rhi.hi.is>
To: J William Piggott <elseifthen@gmx.com>
Cc: util-linux@vger.kernel.org
Subject: Re: [PATCH] doc: howto-man-page.txt: Use font macros instead of font escapes
Date: Thu, 2 Jan 2020 00:17:15 +0000	[thread overview]
Message-ID: <20200102001715.GA30885@rhi.hi.is> (raw)
In-Reply-To: <nycvar.YAK.7.76.1912171449170.1929@zhn.tzk.pbz>

On Tue, Dec 17, 2019 at 03:43:15PM -0500, J William Piggott wrote:
> 
> 
> On Mon, 16 Dec 2019, Bjarni Ingi Gislason wrote:
> 
> > Use font macros instead of font escapes (\f[BIPR]).
> >
[...]
> I wish you luck. I tried to convince this project that inline escapes should be
> avoided, but everyone here seems to be in love with them. I had them all
> stripped out of hwclock(8), but another contributor kept insisting on putting
> them back. I finally gave up and started using them too.
> 

  No reason to give up.
Comment the font-escape line and
add the font-macro line after it.
People need to see the difference easily.

> >
> >###
> > Changes based on:
> >
> > Use a macro to change to the italic font,
> >instead of \fI [1], if possible.
> > The macros have the italic corrections,
> >but "\c" removes the "\/" part.
> >
> > Or
> >
> >add the italic corrections.
> >[1] man-pages(7) [Debian package "manpages"]
> 
> That must be Debian hack, but Michael should adopt a no inline-escape policy
> for the man page project, IMO. Although it shouldn't limited to italic.
> 

  My pointing to reference [1] is wrong,
as there is no instruction about using a macro
instead of a font escape request.

> >
[...]
> >.SH OPTIONS
> >.TP
> >-\fB\-n\fR, \fB\-\-no\-argument\fR
> >+.BR \-n ,\  \-\-no\-argument
> >+.\" \fB\-n\fR, \fB\-\-no\-argument\fR
> 
> Remove the old, don't comment it.
> Same for below.
> 

  Showing the commented out font-escape line is better,
so that the reader sees the difference and
how the transformation is made.
  The commented old line should come first
to prepare the user for the changed (maybe strange) line.

  I find now the use of escaped space (,\ ) worse
than using a quotation (", ").

[...]

-- 
Bjarni I. Gislason

  reply	other threads:[~2020-01-02  0:17 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-16 22:20 [PATCH] doc: howto-man-page.txt: Use font macros instead of font escapes Bjarni Ingi Gislason
2019-12-17 20:43 ` J William Piggott
2020-01-02  0:17   ` Bjarni Ingi Gislason [this message]
2020-01-02 16:52     ` J William Piggott
2020-01-03 11:48       ` Karel Zak

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200102001715.GA30885@rhi.hi.is \
    --to=bjarniig@rhi.hi.is \
    --cc=elseifthen@gmx.com \
    --cc=util-linux@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).