Re: [PATCH 1/3] timespec.3type: tv_nsec is impl-def-type, glibc llong not a bug

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jakub,

On 1/30/23 08:08, Jakub Wilk wrote:
* Alejandro Colomar <alx.manpages@xxxxxxxxx>, 2023-01-29 16:48:
+.BR "    /*\(da*/   tv_nsec;" "  /* Nanoseconds [" 0 ", " 999999999 "] */"

I'm tempted to merge this patch.  It's sooo qute /*↓*/

I want man pages legible, not cute.

I know, I know.

Please make it

   /* see below */ tv_nsec

or maybe

   long /* but see below */ tv_nsec

(given that C23 is not a thing yet).

The reason why I seriously considered /*↓*/ is that it is a bit shocking to the reader, which will prompt it to read the rest of the page to see what the hell that is.

I'm worried that if we make it `long` plus a comment to see below, many will ignore it.

/* see below */ with no mentions to `long` might be a reasonable alternative. Maybe I'd use /* ... */

What do y'all think about it?

Cheers,

Alex



--
<http://www.alejandro-colomar.es/>

Attachment: OpenPGP_signature
Description: OpenPGP digital signature


[Index of Archives]     [Kernel Documentation]     [Netdev]     [Linux Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux