cvs commit: src/sys/compat/linux linux_ioctl.c

Stefan Farfeleder stefan at fafoe.narf.at
Thu Mar 24 13:05:15 PST 2005


On Thu, Mar 24, 2005 at 03:55:59PM -0500, John Baldwin wrote:
> On Thursday 24 March 2005 03:19 pm, Brooks Davis wrote:
> > On Thu, Mar 24, 2005 at 09:09:43PM +0100, Stefan Farfeleder wrote:
> > > On Thu, Mar 24, 2005 at 12:02:10PM -0800, Brooks Davis wrote:
> > > > Prior to C99, I think #error wasn't specified.  C99 does specify it.
> > >
> > > C89/90 had #error too.
> >
> > I stand corrected.  Now that I think about it, all but the perverse
> > preprocessors would implement #error just by blowing up on an unknown
> > directive so even that isn't an issue.  In any case, I've made the
> > change and the new code is at least shorter.  It's really only there for
> > people mucking with kernel internals who should be able to figure out
> > what it means.
> 
> We could always just change CTASSERT() to use #error if that is better.

Unfortunately you can't use preprocessor directives in the replacement
list of a macro.

Stefan


More information about the cvs-src mailing list