djasper added inline comments. ================ Comment at: lib/Format/ContinuationIndenter.cpp:334 @@ +333,3 @@ + if (Style.AlignAfterOpenBracket == FormatStyle::BAS_AlwaysBreak && + Previous.is(tok::l_paren) && State.Column > getNewLineColumn(State) && + (!Previous.Previous || ---------------- klimek wrote: > Is State.Column > getNewLineColumn(State) used to check for the line break? No. It is so that we don't enforce a break when it is not conserving columns, e.g. we'd still do:
f(aaaa, bbbb); http://reviews.llvm.org/D14104 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits