[PATCH] D16113: [clang-tdiy] Add header file extension configuration support.
Haojian Wu via cfe-commits
cfe-commits at lists.llvm.org
Wed Jan 13 16:36:02 PST 2016
hokein added inline comments.
================
Comment at: clang-tidy/tool/ClangTidyMain.cpp:78
@@ +77,3 @@
+static cl::opt<std::string>
+HeaderFileExtensions("header-file-extensions",
+ cl::desc("File extensions that regard as header file.\n"
----------------
alexfh wrote:
> We don't need a command-line flag for this. The regular way to configure clang-tidy is .clang-tidy files. However, if needed, check options can be configured from the command line via the `-config=` option.
>From my understanding, it is a global option for header-file-extensions, right? If we remove this command-line flag, there is only local `header-file-extensions` option remained for particular checks.
Repository:
rL LLVM
http://reviews.llvm.org/D16113
More information about the cfe-commits
mailing list