[squeak-dev] Re: The Trunk: Morphic-ar.305.mcz

Levente Uzonyi leves at elte.hu
Tue Jan 12 06:30:15 UTC 2010


On Mon, 11 Jan 2010, Andreas Raab wrote:

> Levente Uzonyi wrote:
>> On Tue, 12 Jan 2010, commits at source.squeak.org wrote:
>> 
>>> Andreas Raab uploaded a new version of Morphic to project The Trunk:
>>> http://source.squeak.org/trunk/Morphic-ar.305.mcz
>>> 
>>> ==================== Summary ====================
>>> 
>>> Name: Morphic-ar.305
>>> Author: ar
>>> Time: 11 January 2010, 8:05:58.998 pm
>>> UUID: 84c8e065-50b4-f94d-b056-90263e496c28
>>> Ancestors: Morphic-ul.304
>>> 
>>> - Promote isScriptEditorMorph to Object.
>> 
>> Looking at current uses of #isScriptEditorMorph, it seems to be better to 
>> keep it at Morph and add #isMorph checks to the two places which seem to 
>> send it to other objects. This method also seems to be EToys related, so 
>> moving is to the EToys package might be a good idea.
>> Thoughts?
>
> Neither. For one thing, this method already says isScriptEditorMORPH so it 
> obviously implies isMorph == true. (this would be different if it were 
> isScriptEditor because there might be non-morph script editors).

In this case all is*Morph methods should be moved to Object from Morph.

>
> And it clearly can't be moved to the Etoys package - the whole purpose of the 
> method is to make it possibly to unload Etoys and get isScriptEditorMorph == 
> false for all other objects. This would blow up straight into your face if 
> you'd move it to Etoys. Same goes for MorphExtension>>actorState, 
> MorphExtension>>player etc. They all need to stay in the original packages 
> since after unloading Etoys these messages will still be sent; they will just 
> never return true / non-nil respectively.

Of course moving it to EToys would require refactoring of the code.


Levente

>
> Cheers,
>  - Andreas
>
>



More information about the Squeak-dev mailing list