Robert P wrote:

> - (void)dealloc
> {
>   [super dealloc];
>   [undoThing release];
> }

Constructive comment: I think it is recommended that [super dealloc] always be called last in a dealloc: method.

- (void)dealloc
{
  [undoThing release];
  [super dealloc];
}

Bernie