From: Panayotis K. <pan...@pa...> - 2010-07-29 07:17:28
|
On 28 Ιουλ 2010, at 8:44 μ.μ., Arno Puder wrote: > > I just committed your patch. Nice job, Paul! > > Arno > Now that I see the patch again, I have this comment on this part of the code: //- (void)dealloc { // //// TODO is it okay to have this method since this IS currently a category of NSObject? I.e. I want to make sure the real dealloc is called & I can't call super dealloc on NSObject // // objc_setAssociatedObject(self, &memberKey, nil, OBJC_ASSOCIATION_ASSIGN); // [super dealloc]; //} AFAIK the associated object is automatically released when the main object is deallocated. So I think this code is not required at all, I think. |