Word up to Seth and Jerry!
I'll be a Cocoa engineer yet!
-koko
On Sep 21, 2010, at 11:07 PM, Seth Willits wrote:
On Sep 21, 2010, at 9:21 PM, k...@highrolls.net wrote:
NSString *class = [NSString stringWithCString:[data bytes]
encoding:NSASCIIStringEncoding];
Class obstacle = NSClassFromString(class);
id thisObstacle = [[obstacle alloc] initWithImage:[sender
draggedImage] andLocation:[sender draggingLocation]];
[m_obstacles addObject:thisObstacle];
I don't think I have to retain thisObstacle as m_obstacles take
ownership.
No, but you'd better release it after you add it to the array
otherwise you're leaking it. There's no such thing as "taking
ownership" in an exclusive sense. m_obstacles simply retains it when
you added it to the array. But the retain count is already 1 from
the alloc, so now it's at +2. If you remove it from the array it's
at +1 forever. You need to balance that alloc.
However, must I retain [sender draggedImage] so the code becomes:
NSString *class = [NSString stringWithCString:[data bytes]
encoding:NSASCIIStringEncoding];
Class obstacle = NSClassFromString(class);
id thisObstacle = [[obstacle alloc] initWithImage:[[sender
draggedImage] retain] andLocation:[sender draggingLocation]];
[m_obstacles addObject:thisObstacle];
Also no. The obstacle would retain it, if needed, as Jerry said.
--
Seth Willits
_______________________________________________
Cocoa-dev mailing list (Cocoa-dev@lists.apple.com)
Please do not post admin requests or moderator comments to the list.
Contact the moderators at cocoa-dev-admins(at)lists.apple.com
Help/Unsubscribe/Update your Subscription:
http://lists.apple.com/mailman/options/cocoa-dev/koko%40highrolls.net
This email sent to k...@highrolls.net
_______________________________________________
Cocoa-dev mailing list (Cocoa-dev@lists.apple.com)
Please do not post admin requests or moderator comments to the list.
Contact the moderators at cocoa-dev-admins(at)lists.apple.com
Help/Unsubscribe/Update your Subscription:
http://lists.apple.com/mailman/options/cocoa-dev/archive%40mail-archive.com
This email sent to arch...@mail-archive.com
