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

Reply via email to