harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Geir Magnusson Jr <g...@pobox.com>
Subject Re: svn commit: r405120 - /incubator/harmony/enhanced/classlib/trunk/modules/luni/src/main/java/org/apache/harmony/luni/internal/net/www/protocol/http/HttpURLConnection.java
Date Mon, 08 May 2006 22:43:14 GMT


Tim Ellison wrote:
> Geir Magnusson Jr wrote:
>> Why did you get rid of all of the braces, like in
>>
>> if (foo) {
>>   ...
>> }
>>
>> ?
> 
> I did?  Just looked and I don't see that.

Hm.  Sorry - I was scanning through, and I thought that they were 
disappearing.  I figured it was an autoformat that got away from you...

> 
>> I don't want to start code format wars, but I can't understand why you'd
>> want to undo what is considered in many places harmless good practice.
> 
> I agree, my preference is to have the brackets too.
> There are examples of both styles throughout the code.

Ah.  Sorry then.

geir


---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org


Mime
View raw message