Re: [Pharo-project] problem with TextMorph>>handleInteraction:fromEvent:
Hi, is just #handleInteraction: in pharo (my mistake) I'm uploading a changeset. Cheers, Esteban On 2009-07-12 12:58:42 -0300, Stéphane Ducasse <stephane.ducasse-MZpvjPyXg2s@public.gmane.org
said:
in TextMorph I do not see this method On Jul 12, 2009, at 4:25 PM, Esteban Lorenzano wrote: Ouch, bad method, please replace: #handleInteraction:fromEvent: (the squeak event), for #handleInteraction: (the pharo equivallent) Cheers, Esteban On 2009-07-12 11:22:47 -0300, Esteban Lorenzano <estebanlm-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> said: Hi, I'm working to make Algernon works on pharo, and I found what seems to be a bug (or at least something that "revesls" a bug), in TextMorph>>handleInteraction: This is what happens: the Algernon starts and you type a class, then go inside to look for a method, and that sends a keyStroke: to a new TextMorph, who sends a #handleInteraction:fromEvent:. So... old implementation on #handleInteraction:fromEvent: had a first line removed in new implementation: self editor sensor: (KeyboardBuffer new startingEvent: evt). The problem is this: #editor message instances editor attribute if nil. Now... that line is not anymore, introducing a DNU when sending a subsecuent message to editor attribute. I "solved" it bu changing this line: oldEditor := editor. for this other: oldEditor := self editor. (who restores the editor instantiation). What I don't know is: a) is this a bug? b) the bug is there or is a collateral damage? c) my hack really solves the problem? d) is possible to introduce this bugfix into a new version? Cheers, Esteban _______________________________________________ Pharo-project mailing list Pharo-project-bM+ny+RY8h+a+bCvCPl5/gCzwTLBPCX0@public.gmane.org http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Thanks I was doing the same. Can you chekc the next update and let me know? Stef On Jul 12, 2009, at 6:12 PM, Esteban Lorenzano wrote:
Hi, is just #handleInteraction: in pharo (my mistake) I'm uploading a changeset.
Cheers, Esteban
<TextMorphFix.1.cs>
On 2009-07-12 12:58:42 -0300, Stéphane Ducasse <stephane.ducasse-MZpvjPyXg2s@public.gmane.org
said:
in TextMorph I do not see this method
On Jul 12, 2009, at 4:25 PM, Esteban Lorenzano wrote:
Ouch, bad method, please replace: #handleInteraction:fromEvent: (the squeak event), for #handleInteraction: (the pharo equivallent)
Cheers, Esteban
On 2009-07-12 11:22:47 -0300, Esteban Lorenzano <estebanlm-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> said:
Hi, I'm working to make Algernon works on pharo, and I found what seems to be a bug (or at least something that "revesls" a bug), in TextMorph>>handleInteraction: This is what happens: the Algernon starts and you type a class, then go inside to look for a method, and that sends a keyStroke: to a new TextMorph, who sends a #handleInteraction:fromEvent:. So... old implementation on #handleInteraction:fromEvent: had a first line removed in new implementation:
self editor sensor: (KeyboardBuffer new startingEvent: evt).
The problem is this: #editor message instances editor attribute if nil. Now... that line is not anymore, introducing a DNU when sending a subsecuent message to editor attribute. I "solved" it bu changing this line:
oldEditor := editor.
for this other:
oldEditor := self editor.
(who restores the editor instantiation).
What I don't know is:
a) is this a bug? b) the bug is there or is a collateral damage? c) my hack really solves the problem? d) is possible to introduce this bugfix into a new version?
Cheers, Esteban
_______________________________________________ Pharo-project mailing list Pharo-project-bM+ny+RY8h+a+bCvCPl5/gCzwTLBPCX0@public.gmane.org http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Seems to be working :) I'll upload a "pharo compatible" algernon version, then Cheers, Esteban On 2009-07-12 13:29:23 -0300, Stéphane Ducasse <stephane.ducasse@inria.fr> said:
Thanks I was doing the same. Can you chekc the next update and let me know?
Stef
On Jul 12, 2009, at 6:12 PM, Esteban Lorenzano wrote:
Hi, is just #handleInteraction: in pharo (my mistake) I'm uploading a changeset.
Cheers, Esteban
<TextMorphFix.1.cs>
On 2009-07-12 12:58:42 -0300, Stéphane Ducasse <stephane.ducasse-MZpvjP yXg2s@public.gmane.org
said:
in TextMorph I do not see this method
On Jul 12, 2009, at 4:25 PM, Esteban Lorenzano wrote:
Ouch, bad method, please replace: #handleInteraction:fromEvent: (the squeak event), for #handleInteraction: (the pharo equivallent)
Cheers, Esteban
On 2009-07-12 11:22:47 -0300, Esteban Lorenzano <estebanlm-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> said:
Hi, I'm working to make Algernon works on pharo, and I found what seems to be a bug (or at least something that "revesls" a bug), in TextMorph>>handleInteraction: This is what happens: the Algernon starts and you type a class, then go inside to look for a method, and that sends a keyStroke: to a new TextMorph, who sends a #handleInteraction:fromEvent:. So... old implementation on #handleInteraction:fromEvent: had a first line removed in new implementation:
self editor sensor: (KeyboardBuffer new startingEvent: evt).
The problem is this: #editor message instances editor attribute if nil. Now... that line is not anymore, introducing a DNU when sending a subsecuent message to editor attribute. I "solved" it bu changing this line:
oldEditor := editor.
for this other:
oldEditor := self editor.
(who restores the editor instantiation).
What I don't know is:
a) is this a bug? b) the bug is there or is a collateral damage? c) my hack really solves the problem? d) is possible to introduce this bugfix into a new version?
Cheers, Esteban
_______________________________________________ Pharo-project mailing list Pharo-project-bM+ny+RY8h+a+bCvCPl5/gCzwTLBPCX0@public.gmane.org http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project
mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
participants (2)
-
Esteban Lorenzano -
Stéphane Ducasse