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