[Pharo-project] Strange logic in PolygonMorphfromHand: hand

2012-05-13 Thread Stéphane Ducasse
Hi I'm cleaning Sensor use so that we end up with a better layered system. Now I see this method and I wonder (replacing Sensor by hand does not work. But I do not understand why this methods has to explicitly invoke runStepMethods… to me it looks like a broken behavior due probably to the

Re: [Pharo-project] Strange logic in PolygonMorphfromHand: hand

2012-05-13 Thread Fernando Olivero
Indeed is weird. I think it has to explicitly call runStepMethods, because it's not forking , thus taking control of the ui process, does it make sense? I would remove the method (and all the similar ones found in the image, that explicitly reference Sensor and while doing true whileTrue:[ ...],

Re: [Pharo-project] Strange logic in PolygonMorphfromHand: hand

2012-05-13 Thread Stéphane Ducasse
On May 13, 2012, at 8:32 PM, Fernando Olivero wrote: Indeed is weird. I think it has to explicitly call runStepMethods, because it's not forking , thus taking control of the ui process, does it make sense? I would remove the method (and all the similar ones found in the image, that

Re: [Pharo-project] Strange logic in PolygonMorphfromHand: hand

2012-05-13 Thread Sean P. DeNigris
Stéphane Ducasse wrote Now I see this method and I wonder (replacing Sensor by hand does not work. ... What do you think? I'm reimplementing it without Sensor, integrated well with Morphic... It's almost done... I'll try to upload tonight. Sean -- View this message in context:

Re: [Pharo-project] Strange logic in PolygonMorphfromHand: hand

2012-05-13 Thread Igor Stasenko
On 13 May 2012 22:13, Sean P. DeNigris s...@clipperadams.com wrote: Stéphane Ducasse wrote Now I see this method and I wonder (replacing Sensor by hand does not work. ... What do you think? I'm reimplementing it without Sensor, integrated well with Morphic... It's almost done... I'll

Re: [Pharo-project] Strange logic in PolygonMorphfromHand: hand

2012-05-13 Thread Stéphane Ducasse
Sean thanks a lot. pay attention we already did it. Check less sensor bug entry: http://code.google.com/p/pharo/issues/detail?id=5887 Stef On May 13, 2012, at 10:13 PM, Sean P. DeNigris wrote: Stéphane Ducasse wrote Now I see this method and I wonder (replacing Sensor by hand

Re: [Pharo-project] Strange logic in PolygonMorphfromHand: hand

2012-05-13 Thread Sean P. DeNigris
Igor Stasenko wrote You're too slow. We did it already :) WTF!!! You guys are getting too good, lol. One thing... The instance creation should be: poly := PolygonMorph vertices: { p1. p1 } color: Color transparent borderWidth: 2

Re: [Pharo-project] Strange logic in PolygonMorphfromHand: hand

2012-05-13 Thread Igor Stasenko
On 14 May 2012 04:33, Sean P. DeNigris s...@clipperadams.com wrote: Igor Stasenko wrote You're too slow. We did it already :) WTF!!! You guys are getting too good, lol. One thing... The instance creation should be:        poly := PolygonMorph                vertices: { p1. p1 }