From: "Alejandro Colomar (man-pages)" <alx.manpages@gmail.com>
To: "Pali Rohár" <pali@kernel.org>
Cc: "G. Branden Robinson" <g.branden.robinson@gmail.com>,
Michael Kerrisk <mtk.manpages@gmail.com>,
linux-man@vger.kernel.org
Subject: Re: [PATCH v2] termios.3: Add information how to set baud rate to any other value
Date: Sun, 1 Aug 2021 13:07:44 +0200 [thread overview]
Message-ID: <3f5d41da-f7b0-72f3-0104-45bccb18c8a2@gmail.com> (raw)
In-Reply-To: <20210731110649.2irr2eb335jpwuo6@pali>
Hi Pali,
On 7/31/21 1:06 PM, Pali Rohár wrote:
> On Saturday 31 July 2021 13:04:50 Alejandro Colomar (man-pages) wrote:
>> Hi Branden,
>>
>> On 7/31/21 2:19 AM, G. Branden Robinson wrote:
>>> Hi, Pali!
>>>
>>> At 2021-07-30T20:45:36+0200, Pali Rohár wrote:
>>>> +Setting the baud rate to a value other that those defined by
>>>> +.B Bnnn
>>>> +constants is possible via the
>>>
>>> I'd say
>>>
>>> .BI B nnn
>>>
>>> because the "nnn" is not literal, but variable.
>>
>> Agree.
>>
>> But as I already merged the patch, I'll add a new patch with that change.
>> Moreover, man pages mostly-consistently use .B Bnn style (although
>> incorrectly, from reading groff_man(7)), so it deserves a separate patch.
>
> Makes sense. Would you fix style on all places ('git grep Bnn')?
I fixed it right now for Bnnn:
<https://github.com/alejandro-colomar/man-pages/commit/e505393f80d040a03f3c2955e0d236c8e109ee6e>
I'll try to fix it for every other case of FOO* in the pages soon.
Regards,
Alex
--
Alejandro Colomar
Linux man-pages comaintainer; https://www.kernel.org/doc/man-pages/
http://www.alejandro-colomar.es/
prev parent reply other threads:[~2021-08-01 11:07 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-30 15:30 [PATCH] termios.3: Add information how to set baudrate to any other value Pali Rohár
2021-07-30 18:29 ` Alejandro Colomar (man-pages)
2021-07-31 1:50 ` G. Branden Robinson
2021-07-31 11:24 ` Alejandro Colomar (man-pages)
2021-07-30 18:45 ` [PATCH v2] termios.3: Add information how to set baud rate " Pali Rohár
2021-07-30 18:59 ` Alejandro Colomar (man-pages)
2021-07-31 0:19 ` G. Branden Robinson
2021-07-31 11:04 ` Alejandro Colomar (man-pages)
2021-07-31 11:06 ` Pali Rohár
2021-08-01 11:07 ` Alejandro Colomar (man-pages) [this message]
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=3f5d41da-f7b0-72f3-0104-45bccb18c8a2@gmail.com \
--to=alx.manpages@gmail.com \
--cc=g.branden.robinson@gmail.com \
--cc=linux-man@vger.kernel.org \
--cc=mtk.manpages@gmail.com \
--cc=pali@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.