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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.

CU
  Peter



[Index of Archives]     [Linux Media]     [Video 4 Linux]     [Asterisk]     [Samba]     [Xorg]     [Xfree86]     [Linux USB]

  Powered by Linux