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

Reply via email to