Subject: Re: mklocale, take 2
To: None <firstname.lastname@example.org>
From: James Chacon <email@example.com>
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
>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 <firstname.lastname@example.org>