Subject: Re: mklocale, take 2
To: None <thorpej@wasabisystems.com>
From: James Chacon <jchacon@genuity.net>
List: tech-userlevel
Date: 10/18/2001 11:17:22
The original code calls sys/cdefs so I had just left that.

Any porting we want to do to foreign systems will have to provide a cdefs 
anyways.

James

>
>On Thu, Oct 18, 2001 at 03:39:46AM -0400, James Chacon wrote:
>
> > +#include <sys/cdefs.h>
> > +#include <sys/types.h>
>
>Why do you explicitly need <sys/cdefs.h>?  Our style guide says that
><sys/types.h> causes it to be pulled in ... and what in this file
>explicitly relies on it?  Oh, __attribute__ ... well, okay, but this
>is obviously going to be problematic for non-NetBSD hosts.
>
>BTW, our <sys/cdefs.h> should probably #define some sort of pseudo-keyword
>for "packed" ... "packed" is a very important attribute to convey, and it
>should not be deleted if the compiler is not GCC.
>
>-- 
>        -- Jason R. Thorpe <thorpej@wasabisystems.com>
>
>
>
>