[clang] [clang-format][NFC] Simplify AlignMacroMatches (PR #164122)

via cfe-commits cfe-commits at lists.llvm.org
Sat Oct 18 23:42:37 PDT 2025


================
@@ -665,22 +664,21 @@ void WhitespaceManager::alignConsecutiveMacros() {
 
     // If token is a ")", skip over the parameter list, to the
     // token that precedes the "("
-    if (Current->is(tok::r_paren) && Current->MatchingParen) {
-      Current = Current->MatchingParen->Previous;
-      SpacesRequiredBefore = 0;
-    }
-
-    if (!Current || Current->isNot(tok::identifier))
-      return false;
-
-    if (!Current->Previous || Current->Previous->isNot(tok::pp_define))
+    if (const auto *MatchingParen = Current->MatchingParen;
+        Current->is(tok::r_paren) && MatchingParen) {
----------------
owenca wrote:

```suggestion
    if (Current->is(tok::r_paren) && Current->MatchingParen) {
      const auto *MatchingParen = Current->MatchingParen;
```

https://github.com/llvm/llvm-project/pull/164122


More information about the cfe-commits mailing list