Re: [PATCH] sys/types.h: Define new types: [s]nseconds_t

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

Hi наб,

On 12/7/21 14:17, наб wrote:
> On Tue, Dec 07, 2021 at 12:19:58PM +0100, Alejandro Colomar wrote:
>> For symmetry with the existing [s]useconds_t types.
> If you'd hold on to your horses, like, until the next WG14 meeting,
> turns out I know a guy who knows a guy, but my guy is the WG14 project
> editor, and I'm like, a day's away from getting an N number and turns
> out getting your "C defect" paper through is pretty easy if you don't
> have to sit in an NB meeting yourself.
> (Granted, this wouldn't have to be a paper at all if this were C++,
>   but this is pretty good as-is.)

Sounds great! :)

I'll continue with my glibc patch as a draft, to learn how glibc types 
are defined (there's a lot of magic I still need to learn), but will 
wait for the results of your C2X proposal before merging this to glibc.

As Zack suggested, an unsigned version is probably unnecessary, so I'll 
use nsec_t for the patches, as you used in your draft.  I like short 
names, and also signed types.

> 
> You can find my current draft:
>    https://srhtcdn.githack.com/~nabijaczleweli/wg14/blob/868d3e096db0fe4378d77cd67215bed56045196d/N%3F%3F%3F%3F.pdf

It looks good.

Would you mind sharing the source code of your pdf?  I'm curious to 
learn to write those papers.

Also, please CC me when you have any news on that :)

> accepting this would mean just a
>    typedef decltype(struct timespec::tv_nsec) nsec_t;
> to make the implementation C2X conformant again.

Hmm, not sure if it would be better for glibc to define nsec_t in terms 
of timespec, or the other way around.  I had in mind something like the 
following:

Here would go many ifdefs:
typedef /* probably */ long nsec_t;


And this would be clean:
struct timespec {
     ...
     nsec_t tv_nsec;
};


Thanks,
Alex

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