[linux-dvb] Re: [PATCH] LGDT3302 Frontend Driver

[tomi.ollila at iki.fi (Tomi Ollila)]

> Hi,
>
> Yes but why do you need do {} while(0) stuff ?
> You don't need to declare a new block as you add only a if statement, so
>>>+#define dprintk(args...) if (debug) printk(KERN_DEBUG "lgdt3302: " args)
> will be a lot's of cleanner...

what goes wrong below, if this "cleaner" were used

if (something)
	dprintk("message");
else
        do_something_else();