[PATCH] D16113: [clang-tdiy] Add header file extension configuration support.
Haojian Wu via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 22 09:51:40 PST 2016
hokein marked an inline comment as done.
================
Comment at: clang-tidy/utils/HeaderFileExtensionsUtils.cpp:49
@@ +48,3 @@
+ AllHeaderFileExtensions.split(Suffixes, ',');
+ HeaderFileExtensions.clear();
+ for (llvm::StringRef Suffix : Suffixes) {
----------------
I'm +1 on keeping the consistence of `list` option.
I also check the option in clang-tidy checks, seems that this is the first time to introduce list option into clang-tidy. So `;` is fine to me.
http://reviews.llvm.org/D16113
More information about the cfe-commits
mailing list