klimek added inline comments.

================
Comment at: clang-tidy/ClangTidy.cpp:382
@@ +381,3 @@
+      const CommandLineArguments &Args, StringRef Filename) {
+    Context.setCurrentFile(Filename);
+    const ClangTidyOptions &Opts = Context.getOptions();
----------------
My concern is still with the Filename in the arg callback. I'd like to 
understand the full use-case better.
From this code it looks like we want to change the ClangTidyContext's file and 
use the callback for that? That seems rather out of place, but maybe I'm 
missing something.


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