[cfe-commits] [PATCH] 3 small static analyzer patches
Sean McBride
sean at rogue-research.com
Fri Sep 21 08:19:47 PDT 2012
On Thu, 20 Sep 2012 14:02:35 -0700, David Blaikie said:
>The other two patches I'll leave to someone
>more familiar with the Static Analyzer.
OK...
>It might help to resend these with paths that are relative to the
>clang directory, rather than absolute. (eg:
Sorry, that wasn't deliberate... I'll look up the correct svn incantations.
>Also, these patches seem to lack tests. Could you provide tests?
Perhaps... as I said, I was merely perusing the analyzer code out of curiosity I don't really know my way around clang's codebase. But of course searching for 'OSCompareAndSwap' found what looks like a test in tools/clang/test/Analysis/NSString.m.... I don't get the relation to NSString, but I guess I'll add tests for the others there?
Cheers,
--
____________________________________________________________
Sean McBride, B. Eng sean at rogue-research.com
Rogue Research www.rogue-research.com
Mac Software Developer Montréal, Québec, Canada
More information about the cfe-commits
mailing list