[clang] [clang-format[NFC] Replace a function with StringRef::contains (PR #146245)
via cfe-commits
cfe-commits at lists.llvm.org
Sat Jun 28 15:55:19 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang-format
Author: Owen Pan (owenca)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/146245.diff
1 Files Affected:
- (modified) clang/lib/Format/BreakableToken.cpp (+1-13)
``````````diff
diff --git a/clang/lib/Format/BreakableToken.cpp b/clang/lib/Format/BreakableToken.cpp
index def0d73e77539..24912c25ef8c6 100644
--- a/clang/lib/Format/BreakableToken.cpp
+++ b/clang/lib/Format/BreakableToken.cpp
@@ -26,18 +26,6 @@ namespace clang {
namespace format {
static constexpr StringRef Blanks = " \t\v\f\r";
-static bool IsBlank(char C) {
- switch (C) {
- case ' ':
- case '\t':
- case '\v':
- case '\f':
- case '\r':
- return true;
- default:
- return false;
- }
-}
static StringRef getLineCommentIndentPrefix(StringRef Comment,
const FormatStyle &Style) {
@@ -193,7 +181,7 @@ getStringSplit(StringRef Text, unsigned UsedColumns, unsigned ColumnLimit,
if (Chars > MaxSplit || Text.size() <= Advance)
break;
- if (IsBlank(Text[0]))
+ if (Blanks.contains(Text[0]))
SpaceOffset = SplitPoint;
if (Text[0] == '/')
SlashOffset = SplitPoint;
``````````
</details>
https://github.com/llvm/llvm-project/pull/146245
More information about the cfe-commits
mailing list