Re: [PATCH] termios.3: Add information how to set baudrate to any other value

["Alejandro Colomar (man-pages)" <alx.manpages@gmail.com>]

Hi Branden,

On 7/31/21 3:50 AM, G. Branden Robinson wrote:
> Hi, Pali & Alex!
> 
> At 2021-07-30T20:29:24+0200, Alejandro Colomar (man-pages) wrote:
>>>    for the speeds beyond those defined in POSIX.1 (57600 and above).
>>>    Thus, \fBB57600\fP & \fBCBAUDEX\fP is nonzero.
>>
>> I think this should go on a separate paragraph, don't you?
> 
> I would line break after the comma, quote the multiword literal in case
> the bold attribute gets stripped out of the output (copying and
> pasting man page text into emails, for instance), and use font style
> macros instead of font selection escape sequences.

I would too, but that was already existing text that was out of the 
scope of this patch :/

> 
>> .PP?
>>
>>> +Setting baudrate to other value than defined by
>>
>> wfix?:
>>
>> Setting the baud rate to a value other that those defined by
> 
> I'd say "baud rate" as two words in English prose (contrast with C), but
> "than" is correct.

Ouch!  I meant "than"; "that" was a typo that I introduced.  As I 
already applied the patch, I'll fix it myself in my tree before sending 
it to Michael.

Thanks,

Alex

> 
>>> +.B Bnnn
>>> +constants is possible via
>>
>> s/via/via the/?
> 
> I agree.
> 
>>
>>> +.B TCSETS2
>>> +ioctl, see
>>
>> s/,/;/
> 
> I agree.  Comma splices are evil.
> 
> Incidentally, when checking for semantic newline issues, I use the
> following search pattern in Vim.
> 
> /[.;:].
> 
> Strictly, I should say
> 
> /[.;:!?][^\\]
> 
> and were to bind that to a key, that's probably what I'd use.
> 
> Regards,
> Branden
> 


-- 
Alejandro Colomar
Linux man-pages comaintainer; https://www.kernel.org/doc/man-pages/
http://www.alejandro-colomar.es/