================ @@ -628,28 +632,36 @@ class LineJoiner { unsigned tryMergeNamespace(ArrayRef<AnnotatedLine *>::const_iterator I, ArrayRef<AnnotatedLine *>::const_iterator E, - unsigned Limit) { + unsigned Limit, bool OpenBraceWrapped) { if (Limit == 0) return 0; - assert(I[1]); - const auto &L1 = *I[1]; + // The merging code is relative to the opening namespace brace, which could + // be either on the first or second line due to the brace wrapping rules. + const auto OpeningBraceLineOffset = OpenBraceWrapped ? 1 : 0; + const auto BraceOpenLine = I + OpeningBraceLineOffset; ---------------- owenca wrote:
Addressed in 1415aaecb73205547e5a00af93d83ab2a1bcbeec. https://github.com/llvm/llvm-project/pull/123010 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits