[cfe-commits] r163012 - in /cfe/trunk: lib/StaticAnalyzer/Checkers/MallocSizeofChecker.cpp lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp test/Analysis/retain-release.mm

Ted Kremenek kremenek at apple.com
Fri Aug 31 17:05:00 PDT 2012


On Aug 31, 2012, at 1:19 PM, Jordan Rose <jordan_rose at apple.com> wrote:

> Yeah, I had it that way originally, but then I thought that leaving it that way would make it all too easy to make the same mistake the next time we add an exception. This way when I go back with another function to add, I know right away that it should be inside the outer if-statement.
> 
> (I probably should have included braces, though.)

If you want to leave it that way, please add braces.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120831/181682a7/attachment.html>


More information about the cfe-commits mailing list