![\"xcode.jpg\"](\"http:\/\/bobbeaty.com\/wp\/wp-content\/uploads\/2007\/09\/xcode.jpg\" "\\"xcode.jpg\\"")
<\/p>\n<\/p>\n
This afternoon I was working on my latest version of the CryptoQuip solver and decided to turn on the built-in Clang Static Analyzer for all my debug builds as I've gotten to the point that I'm really getting close to having something useful to look at. Interestingly enough, it pointed out a problem on all<\/em><\/strong> my implementations of -copyWithZone:<\/tt> - I was returning an autorelease<\/tt>d instance and that's not the intended signature of the -copyWithZone:<\/tt> method.<\/p>\n For example, my original code for a simple object was:<\/p>\n and the Static Analyzer pointed out that the retain count of +0 was wrong and it was supposed to be +1. I simply changed it to read:<\/p>\n and it's all fixed. But when I'm dealing with a little more complex objects, and I'm making a deep copy, I have to be careful and autorelease<\/tt> the copies I'm making because the setters are calling retain<\/tt> on the arguments, and I don't want to have a leak by leaving a copy with a retain count of +2.<\/p>\n So a more complex version of -copyWithZone:<\/tt> looks like:<\/p>\n where the arguments to the setters are calls to the instance's -copyWithZone:<\/tt> but to get the retain count correct, we need to autorelease<\/tt> them leaving them with a count of +0, to then have retain<\/tt> called on them by the setter.<\/p>\n The NSMutableArray calls retain<\/tt> on all objects that are added to it, and release<\/tt> when they are removed. That explains the call to autorelease<\/tt> on the copy of the individual elements in that deep copy loop.<\/p>\n In general, the Static Analyzer is saving my bacon by avoiding improper code practices as I get back into ObjC coding. I'm really happy with it. Fantastic!<\/p>\n","protected":false},"excerpt":{"rendered":" This afternoon I was working on my latest version of the CryptoQuip solver and decided to turn on the built-in Clang Static Analyzer for all my debug builds as I’ve gotten to the point that I’m really getting close to having something useful to look at. Interestingly enough, it pointed out a problem on all […]<\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"closed","ping_status":"closed","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[9,3],"tags":[],"class_list":["post-3572","post","type-post","status-publish","format-standard","hentry","category-cocoa-coding","category-coding"],"_links":{"self":[{"href":"https:\/\/bobbeaty.com\/wp\/wp-json\/wp\/v2\/posts\/3572","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/bobbeaty.com\/wp\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/bobbeaty.com\/wp\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/bobbeaty.com\/wp\/wp-json\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"https:\/\/bobbeaty.com\/wp\/wp-json\/wp\/v2\/comments?post=3572"}],"version-history":[{"count":5,"href":"https:\/\/bobbeaty.com\/wp\/wp-json\/wp\/v2\/posts\/3572\/revisions"}],"predecessor-version":[{"id":4378,"href":"https:\/\/bobbeaty.com\/wp\/wp-json\/wp\/v2\/posts\/3572\/revisions\/4378"}],"wp:attachment":[{"href":"https:\/\/bobbeaty.com\/wp\/wp-json\/wp\/v2\/media?parent=3572"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/bobbeaty.com\/wp\/wp-json\/wp\/v2\/categories?post=3572"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/bobbeaty.com\/wp\/wp-json\/wp\/v2\/tags?post=3572"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}} \/*!\n This is the standard copy method for the Legend so that we can make\n clean copies without having to worry about all the details. It's a nice\n deep copy, where the contents of the returned Legend are the same as\n this instance's.\n *\/<\/span>\n -<\/span> (<\/span>id<\/span>)<\/span> copyWithZone:<\/span>(<\/span>NSZone<\/span>*<\/span>)<\/span>zone\n {<\/span>\n \/\/ simply use the allocWithZone and populate it properly - easy<\/span>\n return<\/span> [<\/span>[<\/span>[<\/span>Legend allocWithZone:<\/span>zone]<\/span> initWithMap:<\/span>[<\/span>self getMap]<\/span>]<\/span> autorelease]<\/span>;\n }<\/span><\/pre>\n \/*!\n This is the standard copy method for the Legend so that we can make\n clean copies without having to worry about all the details. It's a nice\n deep copy, where the contents of the returned Legend are the same as\n this instance's.\n *\/<\/span>\n -<\/span> (<\/span>id<\/span>)<\/span> copyWithZone:<\/span>(<\/span>NSZone<\/span>*<\/span>)<\/span>zone\n {<\/span>\n \/\/ simply use the allocWithZone and populate it properly - easy<\/span>\n return<\/span> [<\/span>[<\/span>Legend allocWithZone:<\/span>zone]<\/span> initWithMap:<\/span>[<\/span>self getMap]<\/span>]<\/span>;\n }<\/span><\/pre>\n \/*!\n This is the standard copy method for the Quip so that we can make\n clean copies without having to worry about all the details. It's a nice\n deep copy, where the contents of the returned Quip are the same as\n this instance's.\n *\/<\/span>\n -<\/span> (<\/span>id<\/span>)<\/span> copyWithZone:<\/span>(<\/span>NSZone<\/span>*<\/span>)<\/span>zone\n {<\/span>\n \/\/ simply use the allocWithZone and populate it properly - easy<\/span>\n id<\/span> dup =<\/span> [<\/span>[<\/span>Quip allocWithZone:<\/span>zone]<\/span> init]<\/span>;\n \/\/ now let's add in copies of all the ivars we have to make it equal<\/span>\n [<\/span>dup setCypherText:<\/span>[<\/span>[<\/span>[<\/span>self getCypherText]<\/span> copyWithZone:<\/span>zone]<\/span> autorelease]<\/span>]<\/span>;\n [<\/span>dup setStartingLegend:<\/span>[<\/span>[<\/span>[<\/span>self getStartingLegend]<\/span> copyWithZone:<\/span>zone]<\/span>\n autorelease]<\/span>]<\/span>;\n for<\/span> (<\/span>PuzzlePiece*<\/span> pp in<\/span> [<\/span>self getPuzzlePieces]<\/span>)<\/span> {<\/span>\n [<\/span>[<\/span>dup getPuzzlePieces]<\/span> addObject:<\/span>[<\/span>[<\/span>pp copyWithZone:<\/span>zone]<\/span> autorelease]<\/span>]<\/span>;\n }<\/span>\n return<\/span> dup;\t\n }<\/span><\/pre>\n