benhamilton accepted this revision. benhamilton added inline comments. This revision is now accepted and ready to land.
================ Comment at: lib/Format/ContinuationIndenter.cpp:1411 + // line). + if (Current.MatchingParen && Current.MatchingParen->Previous) { + const FormatToken &CurrentScopeOpener = *Current.MatchingParen->Previous; ---------------- Should we check if `State.Stack.back().BreakBeforeParameter` is `true` before doing any of this? Repository: rC Clang https://reviews.llvm.org/D48720 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits