llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-tools-extra Author: Kazu Hirata (kazutakahirata) <details> <summary>Changes</summary> --- Full diff: https://github.com/llvm/llvm-project/pull/139412.diff 1 Files Affected: - (modified) clang-tools-extra/clang-tidy/utils/IncludeSorter.cpp (+2-3) ``````````diff diff --git a/clang-tools-extra/clang-tidy/utils/IncludeSorter.cpp b/clang-tools-extra/clang-tidy/utils/IncludeSorter.cpp index 0fa54b3847ebc..83248e1319484 100644 --- a/clang-tools-extra/clang-tidy/utils/IncludeSorter.cpp +++ b/clang-tools-extra/clang-tidy/utils/IncludeSorter.cpp @@ -19,9 +19,8 @@ namespace { StringRef removeFirstSuffix(StringRef Str, ArrayRef<const char *> Suffixes) { for (StringRef Suffix : Suffixes) { - if (Str.ends_with(Suffix)) { - return Str.substr(0, Str.size() - Suffix.size()); - } + if (Str.consume_back(Suffix)) + return Str; } return Str; } `````````` </details> https://github.com/llvm/llvm-project/pull/139412 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
