Jesse Barnes wrote: > On Fri, 08 May 2009 10:27:49 +0100 > Barry Scott <[email protected]> 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 _______________________________________________ compiz mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/compiz
