[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Garrett Rooney <rooneg_at_electricjellyfish.net>
Date: 2003-04-03 02:35:10 CEST

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

This is an archived mail posted to the Subversion Dev mailing list.