<p dir="ltr"><br>
On 2 Jan 2014 20:06, "Manuel Klimek" <<a href="mailto:klimek@google.com">klimek@google.com</a>> wrote:<br>
><br>
><br>
>   lg.<br>
><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>
> Alexander Kornienko wrote:<br>
> > Manuel Klimek wrote:<br>
> > > Why?<br>
> > I'm not sure what exactly did you mean when writing this, and I was thinking already about asking you, if this comment is still applicable.<br>
> Yea, I don't understand my own fixme any more... Probably we should delete it :)<br>
><br>
> ================<br>
> Comment at: clang-tidy/ClangTidy.h:110<br>
> @@ +109,3 @@<br>
> +  /// \brief Returns an action that runs the specified clang-tidy checks.<br>
> +  virtual FrontendAction *create() LLVM_OVERRIDE;<br>
> +<br>
> ----------------<br>
> Cool. One question is whether it's worth having this small function (with the trivial class) in here at all. Do we expect this to be called outside of clang-tidy itself?</p>
<p dir="ltr">It's already called somewhere in tooling via the FrontendActionFactory interface. What would be a better alternative to this?</p>
<p dir="ltr">><br>
><br>
> <a href="http://llvm-reviews.chandlerc.com/D2481">http://llvm-reviews.chandlerc.com/D2481</a><br>
><br>
> BRANCH<br>
>   svn<br>
><br>
> ARCANIST PROJECT<br>
>   clang-tools-extra<br>
</p>