Re: Overriding -copyWithZone: the right way

FROM : M. Uli Kusterer
DATE : Fri Nov 05 19:31:36 2004

At 17:46 Uhr +0100 05.11.2004, Michael Becker wrote:
>- (id)copyWithZone:(NSZone *)zone {
>    PCShoppingCartCell *copy = [[ PCShoppingCartCell alloc]
>initImageCell:nil];
>    return copy;
>}
>
>This does not crash, but it looks so suspiciously memory-leaking...
>(the alloc/init is not paired with a release on my side). When
>trying to follow the (few) suggestions the docs give me, I tried
>this (but it did not work, the app crashed as soon as the TableView
>wanted to redraw):


  That's just fine. It doesn't leak, because methods with "copy" in
their name, by definition, return retained objects. So, whoever calls
this method knows they're responsible for releasing the object they
get.

  I'm not sure though whether you shouldn't be calling [super
copyWithZone: zone] instead of allocating a new object, and then just
copy over those instance variables your subclass adds to the ones of
the superclass.
--
Cheers,
M. Uli Kusterer
------------------------------------------------------------
        "The Witnesses of TeachText are everywhere..."
                    http://www.zathras.de