On Fri, Feb 11, 2011 at 1:51 PM, <[email protected]> wrote: > Well, I personally like the way it looks: > > this.myPackage = StringInterner.get().intern( > (myPackage.length() == 0) ? "" : (myPackage + '.')); > > becomes: > > this.myPackage = > StringInterner.get().intern( > (myPackage.length() == 0) ? "" : (myPackage + '.')); > > This is going to cause a lot of churn...
I would prefer wrapping assignments only when it is necessary to fit on the line. Ie, this example, the first version is better (and has the advantage of being the way the source is now). The real problem with assignments is when they are really long, such as field initializers, and Eclipse won't break them at all. -- John A. Tamplin Software Engineer (GWT), Google -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
