[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