<p dir="ltr"><br>
On 2 Jan 2014 10:54, "Manuel Klimek" <<a href="mailto:klimek@google.com">klimek@google.com</a>> wrote:<br>
><br>
> ================<br>
> Comment at: clang-tidy/ClangTidy.cpp:156-163<br>
> @@ +155,10 @@<br>
> +    if (AnalyzerChecksEnabled) {<br>
> +      // Run our regex against all possible static analyzer checkers.<br>
> +      // Note that debug checkers print values / run programs to visualize the<br>
> +      // CFG<br>
> +      // and are thus not applicable to clang-tidy in general.<br>
> +      // Always add all core checkers if any other static analyzer checks are<br>
> +      // enabled. This is currently necessary, as other path sensitive checks<br>
> +      // rely<br>
> +      // on the core checkers.<br>
> +      for (unsigned i = 0; i < Checkers.size(); ++i) {<br>
> ----------------<br>
> :gq<br>
><br>
> ================<br>
> Comment at: clang-tidy/ClangTidy.h:107-109<br>
> @@ +106,5 @@<br>
> +///<br>
> +/// FIXME: Ideally we'd want to build a more generic way to use<br>
> +/// \c FrontendAction based checkers in clang-tidy, but that needs some<br>
> +/// preparation work first.<br>
> +class ClangTidyAction : public ASTFrontendAction {<br>
> ----------------<br>
> Why?<br>
><br>
> ================<br>
> Comment at: clang-tidy/ClangTidy.h:110<br>
> @@ +109,3 @@<br>
> +/// preparation work first.<br>
> +class ClangTidyAction : public ASTFrontendAction {<br>
> +public:<br>
> ----------------<br>
> Description says this was moved to make it easier to create the AST consumer, but it doesn't seem to be used outside of the .cpp. What am I missing?</p>
<p dir="ltr">It is supposed to make writing different frontends easier. I've got a patch for an out-of-tree project, that makes use of this.</p>
<p dir="ltr">><br>
><br>
> <a href="http://llvm-reviews.chandlerc.com/D2481">http://llvm-reviews.chandlerc.com/D2481</a><br>
</p>