[PATCH] D14104: clang-format: Add an additional value to AlignAfterOpenBracket: AlwaysBreak.

Daniel Jasper via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 27 03:36:24 PDT 2015


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





More information about the cfe-commits mailing list