Re: svn commit: rev 5521 [...]

On Wednesday, April 2, 2003, at 06:25 PM, Brian Denny wrote:

>> There is an empty line too much above the else.
>
> you mean, you want to see
> if (condition)
> statement;
> else
> statement;
>
> instead of
> if (condition)
> statement;
>
> else
> statement;
>
> ... right? is there a consensus on this, or is it part of our coding
> standard? personally i sometimes like a blank line before an else
> statement, but i don't feel strongly about it.

i think the consensus is that if you are going to add anything extra
(including vertical whitespace), you should be using braces, so it's
clear what's contained in each branch of the if/else.

-garrett

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Apr 3 02:36:03 2003