=?utf-8?q?Björn_Schäpers?= <[email protected]>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/[email protected]>


================
@@ -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
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to