[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