site_archiver@lists.apple.com Delivered-To: cocoa-dev@lists.apple.com Fix this first; I'm surprised it doesn't crash your app almost immediately: On 31 Mar 2010, at 1:52 PM, Dave wrote:
myObjectNameString = [myDictionary objectForKey:[ParserXML parserObjectFieldName]]; myFactYearString = [myDictionary objectForKey:kField_FactYear]; myFactMonthString = [myDictionary objectForKey:kField_FactMonth]; myFactDayString = [myDictionary objectForKey:kField_FactDay]; myFactSourceDatabaseString = [myDictionary objectForKey:kField_FactSourceDatabase]; myFactTextString = [myDictionary objectForKey:kField_FactText]; ... [myObjectNameString release]; [myFactYearString release]; [myFactMonthString release]; [myFactDayString release]; [myFactSourceDatabaseString release]; [myFactTextString release];
Review the memory-management rules. You don't take ownership of any of the objects you get from myDictionary, and you must not release them. If you're using Xcode 3.2 or later, try Build > Build and Analyze from time to time. If you're not, Snow Leopard is only $30. For the rest: If it were me, I'd note that insertRowsAtIndexPaths:withRowAnimation: takes an array, allocate an NSMutableArray and accumulate the paths into it at each iteration of the loop, not doing the insertRows..., reloading the table, or the re-titling of the controller until I was out of the loop. After that, I'd break after the end of the loop, and see if the index-path array contained as many objects as I expected, and that they were all unique. I think I'd have a better picture then. — F _______________________________________________ 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/site_archiver%40lists.apple... This email sent to site_archiver@lists.apple.com