alexfh added a comment.

PTAL


================
Comment at: clang-tidy/ClangTidy.cpp:382
@@ +381,3 @@
+      const CommandLineArguments &Args, StringRef Filename) {
+    ClangTidyOptions Opts = Context.getOptionsForFile(Filename);
+    CommandLineArguments AdjustedArgs;
----------------
The interface should be clearer, if you think about the possibility of having 
different options for different files. Also, the documentation on the 
corresponding methods should help. But adding `getOptionsForFile` doesn't hurt.


http://reviews.llvm.org/D14192



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to