================
Comment at: clang-tidy/tool/ClangTidyMain.cpp:286
@@ -301,3 +285,3 @@
   ClangTidyStats Stats =
       runClangTidy(std::move(OptionsProvider), OptionsParser.getCompilations(),
                    OptionsParser.getSourcePathList(), &Errors,
----------------
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.

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

Reply via email to