================
Comment at: clang-tidy/tool/ClangTidyMain.cpp:286
@@ -301,3 +285,3 @@
ClangTidyStats Stats =
runClangTidy(std::move(OptionsProvider), OptionsParser.getCompilations(),
OptionsParser.getSourcePathList(), &Errors,
----------------
alexfh wrote:
> xazax.hun wrote:
> > My biggest concern so far is that, the list of available checkers are
> > determined dynamically once the compiler is instantiated. For this reason I
> > had to move check listing and some error diagnostic after actually running
> > the tool. This might not be a too big usability issue though, since it was
> > not possible to list the available checkers without input files anyways.
> > However it would be harder to improve on that after this patch is applied.
> A clarification regarding the current state: a file name on the command line
> is currently needed to find a directory to read configuration from. The file
> doesn't have to exist as the analysis isn't run when we just want to list the
> checks.
>
> As for the requirement to run the analysis in order to get the list of
> checks: I don't think it's a good idea. Do you just need this to parse the
> command line?
Yes, I think that I only need the plugin list from the command line. If there
is a nice way to get that without actually running a clang tool that could work
this issue around.
http://reviews.llvm.org/D9555
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
