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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits