nanogui: [patch] fix for zero width/height rectangles


Previous by date: 15 Jul 2005 20:35:26 +0100 Re: [patch] fix for zero width/height rectangles, Aaron J. Grier
Next by date: 15 Jul 2005 20:35:26 +0100 Re: [patch] fix for zero width/height rectangles, Sun Yingfeng
Previous in thread: 15 Jul 2005 20:35:26 +0100 Re: [patch] fix for zero width/height rectangles, Aaron J. Grier
Next in thread: 15 Jul 2005 20:35:26 +0100 Re: [patch] fix for zero width/height rectangles, Sun Yingfeng

Subject: RE: [nanogui] Re: [patch] fix for zero width/height rectangles
From: "Paul Bartholomew" ####@####.####
Date: 15 Jul 2005 20:35:26 +0100
Message-Id: <BAY108-F33F45F5205076E5465A53EEFD00@phx.gbl>

>
>+       if (!w || !h)
>+               return;
>
>wouldn't this be more readable if it was (w == 0 || h == 0)?

This certainly seems like a personal preference to me.  I don't know if 
there are any 'coding standards' in the Microwindows/Nano-X project that 
would define the 'correct' way to represent this...

I'd (personally) use either the "(!w || !h)" version, or:


    ((w == 0) || (h == 0))

(extra parens)

Of course, that's just my personal preference.

- Paul



Previous by date: 15 Jul 2005 20:35:26 +0100 Re: [patch] fix for zero width/height rectangles, Aaron J. Grier
Next by date: 15 Jul 2005 20:35:26 +0100 Re: [patch] fix for zero width/height rectangles, Sun Yingfeng
Previous in thread: 15 Jul 2005 20:35:26 +0100 Re: [patch] fix for zero width/height rectangles, Aaron J. Grier
Next in thread: 15 Jul 2005 20:35:26 +0100 Re: [patch] fix for zero width/height rectangles, Sun Yingfeng


Powered by ezmlm-browse 0.20.