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 ...}.

I really prefer the shorter checks though; not sure why I jumped
through hoops here.

-- 
--
Jesse Barnes, Intel Open Source Technology Center

------------------------------------------------------------------------------
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