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

via cfe-commits cfe-commits at lists.llvm.org
Sun Oct 19 23:23:15 PDT 2025


=?utf-8?q?Björn_Schäpers?= <bjoern at hazardy.de>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/164122 at github.com>


================
@@ -656,7 +656,6 @@ void WhitespaceManager::alignConsecutiveMacros() {
 
   auto AlignMacrosMatches = [](const Change &C) {
     const FormatToken *Current = C.Tok;
----------------
owenca wrote:

This is unique in that a local pointer is initialized to a member of the parameter at the start of the lambda and must remain nonnull till the end after being possibly moved. So the assert would serve as a precondition validation.

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


More information about the cfe-commits mailing list