Eclipse Community Forums: GEF

Home » Eclipse Projects » GEF » wrapping text

Fri, 03 October 2003 05:19

Eclipse User

Originally posted by: ThisisFake.Fakeness.xyz

Is it standard policy to wrap the method calls on the '.'? This is
becomming frustrating when I think I am looking at getCommand() but
actually am looking at i.next().getCommand() which is a method on a
totally different class.

Maybe I need to get use to looking for that, but isin't their a better
cleaner way to wrap text than on the '.'?

Im speaking classes like AbstractEditPart.getCommand(Request);

CL

Re: wrapping text [message #97119 is a reply to message #97104]

Fri, 03 October 2003 10:58

Eclipse User

Wrapping in this fashion is nice because you can put a breakpoint at the
line with getCommand() and not have to step through i.next() to get
there. I think the . should be on the next line, though.

i.next()
.getCommand()

That would make it clear it's not this.getCommand() and I think we do
this in some places.

CL [dnoyeB] Gilbert wrote:
> Is it standard policy to wrap the method calls on the '.'? This is
> becomming frustrating when I think I am looking at getCommand() but
> actually am looking at i.next().getCommand() which is a method on a
> totally different class.
>
> Maybe I need to get use to looking for that, but isin't their a better
> cleaner way to wrap text than on the '.'?
>
> Im speaking classes like AbstractEditPart.getCommand(Request);
>
>
> CL

Re: wrapping editor text [message #97193 is a reply to message #97119]

Fri, 03 October 2003 12:51

Eclipse User

Originally posted by: ThisisFake.Fakeness.xyz

Yes, it sure does make stepping through debugging easier. That is, once
you understand what it means. I think putting the . on the lasterly!?
lastest, rightmost? method will make it more clear.

Glad to hear folks are doing that.

CL

Previous Topic:	Draw2D Layouts
Next Topic:	Encapsulation vs. implementation

Goto Forum:

-=] Back to Top [=-

Current Time: Wed May 07 17:43:42 EDT 2025

.:: Contact :: Home ::.

Breadcrumbs

Sign up to our Newsletter