Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 13 Mar 2004 17:50:50 +0000 (GMT)
From:      Tony Finch <dot@dotat.at>
To:        Ruslan Ermilov <ru@FreeBSD.org>
Cc:        cvs-src@FreeBSD.org
Subject:   Re: cvs commit: src/sys/boot/arc/include arcfuncs.hsrc/sys/boot/i386/boot2 boot2.c src/sys/dev/aic7xxx/aicasm aicasm.c src/sys/dev/cx machdep.h src/sys/dev/ichsmb ichsmb.c src/sys/dev/iir iir.h src/s
Message-ID:  <Pine.SOL.4.44.0403131749590.12445-100000@orange.csi.cam.ac.uk>
In-Reply-To: <20040312223315.GE4588@ip.net.ua>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, 13 Mar 2004, Ruslan Ermilov wrote:
> On Fri, Mar 12, 2004 at 01:45:45PM -0800, Tom Rhodes wrote:
> >
> > -#ifdef __GNUC__
> > -#if __GNUC__ >= 2
> > +#if defined(__GNUC__) || defined(__INTEL_COMPILER)
> > +#if __GNUC__ >= 2 || defined(__INTEL_COMPILER)
> >  #pragma pack(4)
> >  #endif
> >  #endif
>
> These ifdefs are broken.
>
> #if (defined(__GNUC__) && __GNUC >= 2) || defined(__INTEL_COMPILER)
>
> would be more correct.

Why, given that an undefined macro is equivalent to 0 in this context?

-- 
Tony Finch  <dot@dotat.at>  http://dotat.at/



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.SOL.4.44.0403131749590.12445-100000>