On Aug 31, 2012, at 13:18 , Ted Kremenek <[email protected]> wrote:
> > On Aug 31, 2012, at 11:19 AM, Jordan Rose <[email protected]> wrote: > >> + if (Name) >> + if (Name->isStr("CGBitmapContextCreateWithData")) >> + RE = S->getRetEffect(); >> } > > This can just be "if (Name && Name->isStr(…))" 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.) _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
