Jesse Barnes wrote:
> On Fri, 08 May 2009 10:27:49 +0100
> Barry Scott <barry.sc...@onelan.co.uk> wrote:
>
>   
>> Jesse Barnes wrote:
>>
>> Why are you comparing booleans to TRUE and worse then negating the
>> result?
>>
>> if( !noWait )
>>
>> Is clearer.
>>
>> -                    waitForVideoSync (s);
>> +                    if (!(noWait == TRUE))
>> +                        waitForVideoSync (s);
>>     
>
> Yeah I've been looking at too much X code.  In some places the style is
> like if (foo == NULL) { ... handle failure ...}.
>   
Personally I always compare to NULL if foo is a pointer as it makes the 
intent
of the code clear.

Barry


------------------------------------------------------------------------------
The NEW KODAK i700 Series Scanners deliver under ANY circumstances! Your
production scanning environment may not be a perfect world - but thanks to
Kodak, there's a perfect scanner to get the job done! With the NEW KODAK i700
Series Scanner you'll get full speed at 300 dpi even with all image 
processing features enabled. http://p.sf.net/sfu/kodak-com
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to