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