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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to