I can't find a way to weight the line breaks, to make it resort to assignments last. I'll drop this one.
On Fri, Feb 11, 2011 at 10:57 AM, John Tamplin <[email protected]> wrote: > 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
