Hi clang,<br><br>I fixed upper issues, and here is the new patch.<br><br><div class="gmail_quote">2011/5/4 Ted Kremenek <span dir="ltr"><<a href="mailto:kremenek@apple.com">kremenek@apple.com</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
One other thing I noticed in the patch:<br>
<br>
  (const Decl* D, AnalysisManager &mgr, ...)<br>
<br>
why use a generic Decl*?  Since it is always going to be a TranslationUnitDecl*, we should pass the most specific type.<br>
<div><div></div><div class="h5"><br>
On May 3, 2011, at 10:50 AM, Argyrios Kyrtzidis wrote:<br>
<br>
> A more appropriate name is checkEndOfTranslationUnit (a checker can already check the TranslationUnitDecl with checkASTDecl which is called at the beginning).<br>
><br>
> So please change runCheckersOnTranslationUnit -> runCheckersOnEndOfTranslationUnit and likewise for the rest of the patch ("class TranslationUnit" -> "class EndOfTranslationUnit", etc.).<br>

><br>
> Otherwise looks good!<br>
><br>
> On May 2, 2011, at 8:33 PM, ียภฺ wrote:<br>
><br>
>> Hi Clang,<br>
>><br>
>> This patch add a new check position checkTranslationUnit for a check after all the decls in the translation unit analyzed, then we can do some statistical analysis in this check position.<br>
>><br>
>> This patch is preparation for statistical UncheckedRenturn checker.<br>
>> I'll appreciate it if there are any advice about this patch.<br>
>><br>
>> --<br>
>> Best regards!<br>
>><br>
>> Lei Zhang<br>
>> <checkTranslationUnit.patch>_______________________________________________<br>
>> cfe-commits mailing list<br>
>> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Best regards!<br><br>Lei Zhang<br>