Short answer: Yes. Long answer: The implementation in BorderedMorph was fixed in Polymorph-Overrides, then that version was promoted to Morph, then BorderedMorph reverted to pre-Polymorph-Overrides functionality from the removeSelector: in my cs. So the "correct" thing to do would be to remove all versions of the method from BorderedMorph, as it's former (correct) version is now used by Morph.
Cheers, Henry Stéphane Ducasse wrote: > so henrik > is is ok if I remove areasRemainingToFill: from BorderedMorph? > > Stef > > On Mar 19, 2009, at 1:46 PM, Henrik Johansen wrote: > > >> I know, which is why I thought it was strange, the .cs contained: >> !BorderedMorph removeSelector: #areasRemainingToFill:! >> which when I file it in, results in no areasRemainingToFill: method >> for >> BorderedMorph in the image. >> >> I really have no idea how the update process makes it revert to an >> older >> version instead, nor how one would go about fixing it... :/ >> >> Cheers, >> Henrik >> >> Stéphane Ducasse wrote: >> >>> henrik I just integrated the file you gave me. >>> If you have a new version that fixes the problem I will do a new >>> version. >>> >>> Stef >>> >>> On Mar 19, 2009, at 12:20 PM, Henrik Johansen wrote: >>> >>> >>>> It seems BorderedMorph's areasRemainingToFill: method got reverted >>>> to >>>> the version in use prior to the override by PolyMorph (which has now >>>> been promoted to Morph), instead of removed completely. >>>> (This is the state my image is in after updating a released 248, at >>>> least) >>>> >>>> Cheers, >>>> Henry >>>> >>>> Stéphane Ducasse wrote: >>>> >>>>> Issue 655: Overlapping Graphics on World do not Properly Repaint >>>>> Issue 658: Preferences resetHaloSpecifications >>>>> Issue 664: unimplemented errorCantGoBack >>>>> Closed 665 >>>>> >>>>> now we will focus on the closures. >>>>> >>>>> Stef >>>>> >>>>> >>>>> >>>>> >>>>> >>>>> _______________________________________________ >>>>> Pharo-project mailing list >>>>> [email protected] >>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>> >>>>> >>>>> >>>>> >>>> _______________________________________________ >>>> Pharo-project mailing list >>>> [email protected] >>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>> >>>> >>> ------------------------------------------------------------------------ >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> [email protected] >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> _______________________________________________ >> Pharo-project mailing list >> [email protected] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> >> > > > _______________________________________________ > Pharo-project mailing list > [email protected] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ Pharo-project mailing list [email protected] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
