[Newbies] Deleting unopened morphs after unit tests

Bert Freudenberg bert at freudenbergs.de
Wed Jun 16 22:45:16 UTC 2010


On 17.06.2010, at 00:02, Sean P. DeNigris wrote:

> 
> http://wiki.squeak.org/squeak/428 says:
>  When you create Morphs, even if you do not do an openInWorld, you should
> delete them.
>    ...
>    tearDown
>      aPatchArchiveClient delete
> 
> However, the method comment for Morph>>delete is:
> 	"Remove the receiver as a submorph of its owner and make its 
> 	new owner be nil."
> 
> If the morph was not opened in the world, won't the owner of the top-level
> morph already be nil?  I want to update either the wiki or the comment to be
> in sync, but I'm not sure which is right.

Look for implementers of #delete - quite a few Morph subclasses override it to do some cleanup.

So the advice to call it is good.

- Bert -




More information about the Beginners mailing list