Michael Krufky wrote:
>
> > > +#define dprintk(args...) \
> > > +do { \
> > > +if (debug) printk(KERN_DEBUG "lgdt3302: " args); \
> > > +} while (0)
> >
> > 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...
>
> Agreed... I will add make that change.
Better not. Code like this will brake horrible without the do-while:
if (foo)
dprintk(...);
else
bar();
Ciao, ET.
