Question

I’m fairly new to Cocoa and Objective-C. Currently I’m developing a fairly basic application to test my knowledge and put some of the stuff I’ve been reading about into practice. Everything is working, but Leaks reports a number of issues.

None of these leaks seems to be directly applicable to code that I’ve written (I have read and tried to follow Apple’s rules on memory allocation). Currently my project makes use of Garbage Collection and I’m developing on Snow Leopard. Running AnalysisTool finds no issues with my code (aside from a few naming convention warnings).

Currently my application makes use of an NSTableView which I have hooked up to an NSArrayController. Interacting with the NSTableView seems to cause leaks to report issues (actions such as sorting table columns and other standard user interaction). This leads me to believe that my use of the NSArrayController (and my implementation of its content source) is to blame.

Currently the NSArrayController receives its content from an NSMutableArray (timers) handled in my Application’s delegate like so:

- (id) init
{
    if (self = [super init])
    {
        timers = [NSMutableArray array];    
    }
    return self;
}
- (void) dealloc
{
    [timers release];
    [super dealloc];
}

Within Interface Builder my NSArrayController has its Object Controller set to the Timing class, which is defined below:

@interface Timing : NSObject {
    NSString *desc;
    NSDate *timestamp;
    bool active;
}
@end

@implementation Timing
-(id) init
{
    if (self = [super init])
    {
        desc = @"New timing";
        timestamp = [[NSDate alloc] init];
        active = false;
    }
    return self;
}
-(void) dealloc
{
    [timestamp release];
    [super dealloc];
}
@end

I’ve used standard Cocoa bindings to hook up Add and Remove buttons to manipulate the TableView and these seem to work correctly (clicking Add will create a row in the TableView with the value of ‘New timing’, for instance).

Leaks reports that the libraries responsible are AppKit and CoreGraphics. Although, honestly, I’m still new to the Leaks tool – so I could be reading its output incorrectly. If it helps, I’ve placed a screenshot of its output here. If anyone could point me in the right direction, that would really be appreciated.

As an aside, I’ve also been experimenting with manually adding objects to the timers array without the use of Cocoa bindings. Here’s what I came up with:

Timing *timingInstance = [[Timing alloc] init];
[timers addObject:timingInstance];
[timingInstance release];

[timersController setContent:timers];
[timersTableView reloadData];

Again, this seems to work, but I thought it best to ask the experts!

Answer 1

Your memory management for the timers array is not quite correct. Using the array factory method will return an instance of NSMutableArray that has already been autoreleased, so the lifetime of that object is (probably) limited to the end of the current run loop, and it will be over-released when you call release in your dealloc method. The proper way to do it is as follows:

- (id) init
{
    if (self = [super init])
    {
        timers = [[NSMutableArray alloc] initWithCapacity:0];
    }
    return self;
}

This method will give you an instance of NSMutableArray with a retain count of 1, which will then drop to zero (and properly release the memory) when you call release in your dealloc method. The call to alloc in your init method is balanced out by the call to release in your dealloc method. I notice that this is the exact pattern that you used for your NSDate object in the Timing class, so you are already familiar with the idea.