Heh, I agree with everything, except #2. I prefer the arguments to line up at the first one. I can live with the other approach though.
On 30 Oct 2007, at 02:25, Alex Miller wrote: > I agree with Jason and Eugene > > ----- Original Message ----- > From: "Jason Voegele" <[EMAIL PROTECTED]> > To: [email protected] > Sent: Monday, October 29, 2007 7:40:42 PM (GMT-0600) America/Chicago > Subject: Re: [tc-dev] eclipse .classpath, etc > > On Monday 29 October 2007 7:41:59 pm Eugene Kuleshov wrote: >> -- don't align field declarations by columns (it look really ugly and >> harder to read when there are mix of fields with short and long >> names, >> i.e. constants) >> -- don't indent contructor, method call params and few other >> constructs >> on column (it also look quite bad wen parameters starts far from >> the left) >> -- don't format line and block comments and don't indent those >> comments >> either > > I'm with Eugene on these, most especially item #2. I find it > especially ugly > to have a column of method/constructor parameters shoved against > the right > margin with huge blocks of whitespace to the left. If a parameter > list does > need to extend over more than one line, I much prefer just having > subsequent > lines indented two levels. > > But then again, this probably as decidable as "Vi or Emacs" or > "where should > one place one's curly braces". :-) > > -- > Jason Voegele > Don't get mad, get interest. > _______________________________________________ > tc-dev mailing list > [email protected] > http://lists.terracotta.org/mailman/listinfo/tc-dev > > _______________________________________________ > tc-dev mailing list > [email protected] > http://lists.terracotta.org/mailman/listinfo/tc-dev -- Geert Bevin Terracotta - http://www.terracotta.org Uwyn "Use what you need" - http://uwyn.com RIFE Java application framework - http://rifers.org Music and words - http://gbevin.com _______________________________________________ tc-dev mailing list [email protected] http://lists.terracotta.org/mailman/listinfo/tc-dev
