Login | Register
My pages Projects Community openCollabNet

Discussions > dev > PathConv.paint() method - should have Graphics argument.

gef
Discussion topic

Hide all messages in topic

All messages in topic

Re: [gef-dev] PathConv.paint() method - should have Graphics argument.

Reply

Author bobtarling
Full name Bob Tarling
Date 2008-11-23 06:53:09 PST
Message I'm moments away from the build so have just done this.

Bob.


2008/11/23 Bob Tarling <bob dot tarling at gmail dot com>:
> Opps how dumb of me.
>
> Yes, please adjust PathItemPlacementStrategy and any PathConv classes
> that implement it.
>
> Don't worry about deprecation of methods like this that have no made a
> stable release yet.
>
> If this isn't done by the time I create a milestone I'll make sure I
> do this myself first.
>
> Bob.
>
>
> 2008/11/22 Dave Thompson <argouml at davet dot org>:
>> Hi,
>>
>> The empty PathConv.paint() method which currently exists in
>> org.tigris.gef.base.PathConv has no argument so it is not possible to pass a
>> Graphics object for it to paint to. This makes it different to the other
>> 143 paint methods in GEF, which all have a Graphics argument.
>>
>> If it is ok, I will deprecate paint() and add a new empty method
>> paint(Graphics g).
>>
>> Please tell me if this is the wrong thing to do, because it would mean that
>> I've missed something.
>>
>> Regards,
>>
>> Dave
>>
>> --------------------​--------------------​--------------------​---------
>> To unsubscribe, e-mail: dev-unsubscribe at gef dot tigris dot org
>> For additional commands, e-mail: dev-help at gef dot tigris dot org
>>
>>
>

Re: [gef-dev] PathConv.paint() method - should have Graphics argument.

Reply

Author bobtarling
Full name Bob Tarling
Date 2008-11-23 06:15:50 PST
Message Opps how dumb of me.

Yes, please adjust PathItemPlacementStrategy and any PathConv classes
that implement it.

Don't worry about deprecation of methods like this that have no made a
stable release yet.

If this isn't done by the time I create a milestone I'll make sure I
do this myself first.

Bob.


2008/11/22 Dave Thompson <argouml at davet dot org>:
> Hi,
>
> The empty PathConv.paint() method which currently exists in
> org.tigris.gef.base.PathConv has no argument so it is not possible to pass a
> Graphics object for it to paint to. This makes it different to the other
> 143 paint methods in GEF, which all have a Graphics argument.
>
> If it is ok, I will deprecate paint() and add a new empty method
> paint(Graphics g).
>
> Please tell me if this is the wrong thing to do, because it would mean that
> I've missed something.
>
> Regards,
>
> Dave
>
> --------------------​--------------------​--------------------​---------
> To unsubscribe, e-mail: dev-unsubscribe at gef dot tigris dot org
> For additional commands, e-mail: dev-help at gef dot tigris dot org
>
>

PathConv.paint() method - should have Graphics argument.

Reply

Author dthompson
Full name Dave Thompson
Date 2008-11-22 13:16:26 PST
Message Hi,

The empty PathConv.paint() method which currently exists in
org.tigris.gef.base.PathConv has no argument so it is not possible to pass a
Graphics object for it to paint to. This makes it different to the other 143
paint methods in GEF, which all have a Graphics argument.

If it is ok, I will deprecate paint() and add a new empty method
paint(Graphics g).

Please tell me if this is the wrong thing to do, because it would mean that
I've missed something.

Regards,

Dave
Messages per page: