hokein marked 3 inline comments as done. ================ Comment at: clang-tidy/utils/HeaderFileExtensionsUtils.cpp:51 @@ +50,3 @@ + HeaderFileExtensions.clear(); + for (llvm::StringRef Suffix : Suffixes) { + llvm::StringRef Extension = Suffix.trim(); ---------------- Thanks for the explanation. Now I add `delimiter` parameter to this function to allow developers to set specified character.
================ Comment at: clang-tidy/utils/HeaderFileExtensionsUtils.h:34 @@ +33,3 @@ +bool isPresumedLocInHeaderFile( + SourceLocation Loc, SourceManager &SM, + const HeaderFileExtensionsSet &HeaderFileExtensions); ---------------- It is used in `UnnamedNamespaceInHeaderCheck`. http://reviews.llvm.org/D16113 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits