[PATCH v4 1/3] bits/sem.h: Split out struct semid_ds

Alistair Francis alistair23@gmail.com
Thu Mar 26 21:58:14 GMT 2020


On Thu, Mar 26, 2020 at 11:59 AM Joseph Myers <joseph@codesourcery.com> wrote:
>
> On Thu, 26 Mar 2020, Alistair Francis via Libc-alpha wrote:
>
> > Split out the struct semid_ds into it's own file. This will allow us to
> > have architectures specify their own version.
> > ---
> >  sysdeps/unix/sysv/linux/Makefile          |  1 +
> >  sysdeps/unix/sysv/linux/bits/sem.h        | 24 +------------
> >  sysdeps/unix/sysv/linux/bits/semid_ds_t.h | 43 +++++++++++++++++++++++
>
> That's not the naming convention.  A header that just defines a given type
> for use in other installed headers is bytes/types/<type_name>.h.  So
> bits/types/struct_semid_ds.h (there's no _t on this type name).

Fixed!

Alistair

>
> --
> Joseph S. Myers
> joseph@codesourcery.com


More information about the Libc-alpha mailing list