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

[Michael Krufky]

Peter Beutner wrote:

> Ben Dooks schrieb:
>
>>> I dont see why the code you posted above will break with dprintk 
>>> defined like this:
>>> #define dprintk(args...) if (debug) printk(KERN_DEBUG "lgdt3302: " 
>>> args)
>>>
>>> Imo:
>>>
>>> if(foo)
>>>     if (debug) printk(KERN_DEBUG "lgdt3302: " args);
>>> else
>>>     bar();
>>>
>>> should work like intended.
>>
>>
>>
>> The else will be part of the if (debug), not the if (foo) as
>> intended. This means that changing the dprintk macro would
>> end up breaking a number of things. Best to avoid these problems
>> in the first place.
>>
> I see. Thx for the explanation.

Doesn't this cover all bases:

  #define dprintk(args...) { if (debug) printk(KERN_DEBUG "lgdt3302: " 
args); }

???

-- 
Michael Krufky