German, did you receive any answer?

Diego

> Hi everybody,
>
> Following the 1st step in the Morph Cleaning Project (small SLint
> suggested refactorings), I was trying the "Messages sent but not
> implemented" SLint rule only on Morph.
>
> I came across some interesting results, for example things that
> shouldn't work as they are, or perhaps work but only on some Morph
> subclasses or given some context.
>
> I'm interested in comments from experienced people on why this things
> are implemented this way - perhaps there are obscure reasons out of my
> reach.
>
> Here is the list:
>
> #addStandardHaloMenuItemsTo:hand:
> sends: #addWorldHaloMenuItemsTo:hand:
> which is defined in: PasteUpMorph
> comment: checks for #isWorldMorph before sending
> solution: move part to PasteUpMorph and call super
>
> #beThisWorldsModel
> sends: #model:slotName:
> which is defined in: MorphicModel
> comment: checks for #isMorphicModel before sending
> solution: move beThisWorldsModel to MorphicModel; add some mechanism to
> #buildMetaMenu: so subclasses can add menu options. the menu options
> will probably be mixed.
>
> #buildDebugMenu:
> sends: #model
> which is defined in: MorphicModel
> comment: checks for #isMorphicModel before sending
> solution: add some mechanism to #buildDebugMenu: so subclasses can add
> menu options. the menu options will probably be mixed.
>
> #chooseNewGraphicCoexisting:
> sends: #form
> which is defined in: some Morph subclasses
> comment: doesn't work for some Morph subclasses
> solution: welcome
>
> #choosePartName
> sends: #nameMeIn: #renameMe
> which are defined in: Component, ComponentLikeModel
> comment: checks for #self world model isKindOf: Component before
> sending solution: welcome
>
> #convertAugust1998:using:
> sends: #bePossessive
> which is defined in: TileMorph
> comment: checks for key == #possessive before sending
> solution: welcome
>
> #enforceTileColorPolicy
> sends: #coloredTilesEnabled
> which is defined in: nowhere
> comment: relies on #doesNotUnderstand and #valueOfFlag:
> solution: not needed
>
> #removeAlarm:at:
> sends: #removeAlarm:at:for:
> which is defined in: nowhere
> comment: it shouldn't work
> solution: welcome
>
> #showActions
> sends: #actionSelector
> which is defined in: various Morph subclasses
> comment: checks if respondsTo: #actionSelector before sending
> solution: not needed
>
> #specialNameInModel
> sends: #slotName
> which is defined in: MorphicModel and others
> comment: checks for #isMorphicModel before sending
> solution: move part to MorphicModel and call super
>
> #undoMove:redo:owner:bounds:predecessor:
> sends: #activate
> which is defined in: SystemWindow
> comment: checks for #isSystemWindow before sending
> solution: move part to SystemWindow and call super
>
>
> Thanks in advance,
>
> German Morales