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
