On Fri, Jul 3, 2009 at 5:53 AM, <[email protected]> wrote: > - if (-1 == (int) Res || ! Res) > - { > - return Res; > + if (-1 == (int) Res || !Res) > + { > + return FALSE;
I know you didn't write the check and were just changing the return values but as a matter of style could we do something like if (((int) Res || !Res) == -1) or would the extra parens mess up the order of operations? Not a big deal, its just easier on my eyes. Thanks -- Steven Edwards "There is one thing stronger than all the armies in the world, and that is an idea whose time has come." - Victor Hugo _______________________________________________ Ros-dev mailing list [email protected] http://www.reactos.org/mailman/listinfo/ros-dev
