Re: Fix empty macros in acpi.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


On Tue, Jun 12, 2007 at 08:21:15PM -0400, Dave Jones wrote:
> On Wed, Jun 13, 2007 at 01:00:29AM +0100, Al Viro wrote:
>  > On Tue, Jun 12, 2007 at 07:33:09PM -0400, Dave Jones wrote:
>  > > +#define DBG(x...) do { } while(0)
>  > 
>  > Eh...  Please, stop it - if you want a function-call-like no-op returning void,
>  > use ((void)0).  At least that way one can say DBG(....),foo(), etc.
> They both end up compiled to nothing anyway, so I'm not bothered
> either way..   I'm not sure I follow why the syntax of that last part
> is a good thing.  It looks like something we'd want to avoid rather
> than promote?

If on one side of ifdef it's a void-valued expression, so it should be
on another; the reason is that we don't get surprise differences between
the builds...

IOW, if it doesn't build in some context, it should consistently fail to
build in that context.
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at
Please read the FAQ at

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux