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

Tomi Ollila tomi.ollila at iki.fi
Tue Jun 28 15:25:52 CEST 2005


> 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();





More information about the linux-dvb mailing list