From owner-cvs-src@FreeBSD.ORG Thu Mar 24 21:08:26 2005 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 56C9A16A4CF; Thu, 24 Mar 2005 21:08:26 +0000 (GMT) Received: from critter.freebsd.dk (f170.freebsd.dk [212.242.86.170]) by mx1.FreeBSD.org (Postfix) with ESMTP id 394F943D64; Thu, 24 Mar 2005 21:08:25 +0000 (GMT) (envelope-from phk@critter.freebsd.dk) Received: from critter.freebsd.dk (localhost [127.0.0.1]) by critter.freebsd.dk (8.13.3/8.13.1) with ESMTP id j2OL8JVT003276; Thu, 24 Mar 2005 22:08:19 +0100 (CET) (envelope-from phk@critter.freebsd.dk) To: Stefan Farfeleder From: "Poul-Henning Kamp" In-Reply-To: Your message of "Thu, 24 Mar 2005 22:05:08 +0100." <20050324210504.GL46915@wombat.fafoe.narf.at> Date: Thu, 24 Mar 2005 22:08:19 +0100 Message-ID: <3275.1111698499@critter.freebsd.dk> Sender: phk@critter.freebsd.dk cc: Brooks Davis cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org cc: John Baldwin Subject: Re: cvs commit: src/sys/compat/linux linux_ioctl.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 24 Mar 2005 21:08:26 -0000 In message <20050324210504.GL46915@wombat.fafoe.narf.at>, Stefan Farfeleder wri tes: >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. And you can't use all sorts of repressions in #if either. that's why CTASSERT() is such a magic beast. -- Poul-Henning Kamp | UNIX since Zilog Zeus 3.20 phk@FreeBSD.ORG | TCP/IP since RFC 956 FreeBSD committer | BSD since 4.3-tahoe Never attribute to malice what can adequately be explained by incompetence.