[PATCH] D14192: Add ExtraArgs and ExtraArgsBefore options to enable clang warnings via configuration files.

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 6 07:18:33 PST 2015


alexfh 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();
----------------
klimek wrote:
> 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.
We need the filename to retrieve the proper ClangTidyOptions for each file. Context.setCurrentFile is used to specify which file should getOptions return the options for.


http://reviews.llvm.org/D14192





More information about the cfe-commits mailing list