Source-Changes-D archive

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

Re: CVS commit: src/common/lib/libc/string



On Mon, Aug 22, 2011 at 04:48:02PM +0400, Valeriy E. Ushakov wrote:
> > >  > It doesn't. The prototypes in strings.h already ensure that
> > >  > uint32_t/uint64_t are present and that's the only thing it could ever
> > >  > need from stdint.h.
> > > 
> > > Yes it does. strings.h is included by string.h and is therefore not
> > > allowed to include stdint.h itself.
> > 
> > You are skinning the wrong cat. string.h shouldn't include strings.h in
> > first place.
> 
> #if defined(_NETBSD_SOURCE)
> #include <strings.h>          /* for backwards-compatibilty */
> ...

I'm aware of that. It was pointed out the last time this behavior was
*properly* discussed. I wouldn't mind killing that part completely, but
I completely disagree with David's changed for strings.h.

Joerg


Home | Main Index | Thread Index | Old Index