Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 13 Nov 2002 13:47:34 -0800
From:      Maxime Henrion <mux@freebsd.org>
To:        Archie Cobbs <archie@dellroad.org>
Cc:        cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG
Subject:   Re: cvs commit: src/sys/sys stddef.h
Message-ID:  <20021113214734.GC12995@elvis.mu.org>
In-Reply-To: <200211131832.gADIWRDO022590@arch20m.dellroad.org>
References:  <200211131514.gADFEv54035510@repoman.freebsd.org> <200211131832.gADIWRDO022590@arch20m.dellroad.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Archie Cobbs wrote:
> Maxime Henrion wrote:
> >   Added files:
> >     sys/sys              stddef.h 
> >   Log:
> >   Add the sys/stddef.h header, so that we can have ptrdiff_t
> >   definition in the kernel.  It also includes the offsetof()
> >   definition which, according to bde@ should be here and not
> >   in sys/types.h, and the definition of NULL so that we're
> >   closer to make it closer to standard C.
> 
> FYI,
> 
> There are lots of existing definitions of NULL in the source tree:
> 
>     src/include/dirent.h
>     src/include/locale.h
>     src/include/stddef.h
>     src/include/stdio.h
>     src/include/stdlib.h
>     src/include/string.h
>     src/include/time.h
>     src/include/unistd.h
>     src/include/wchar.h
>     src/include/rpc/types.h
>     src/lib/libstand/stand.h
>     src/sys/boot/libstand/stand.h
>     src/sys/sys/param.h
> 
> Might be nice to consolidate some of them with your new definition
> in stddef.h.

Yeah, I'm aware of these problems.  I'll try to take care of those,
though my TODO list is fairly full right now.  I needed sys/stddef.h for
the ptrdiff_t definition, in order to implement %t.  Also, there is
another definition of offsetof() in sys/types.h which should be gotten
rid of.  If someone with more time on his hands can handle this job, it
would be very nice.

Cheers,
Maxime

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20021113214734.GC12995>