[PATCH] D55470: Cpp11 Braced List Style Ignores BAS_AlwaysBreak

[John Gehrig via Phabricator via cfe-commits]

jgehrig created this revision.
jgehrig added a project: clang.
Herald added a subscriber: cfe-commits.

First patch attempt :)

I've noticed that clang-format does not honor the BAS_AlwaysBreak when Cpp11BracedListStyle is set. This seems like a bug, as BAS_DontAlign will change the line continuation behavior.

This patch makes BAS_AlwaysBreak affect the line continuation behavior in the same way it does with functions or constructors.

Unit Tests should demonstrate the proposed modifications. I also have two related patches which I intend to propose: spaces inside Cpp11BracedList {}s, and parameterization of the 'magic number' which overrides BinPackArguments for Cpp11BracedLists.


Repository:
  rC Clang

https://reviews.llvm.org/D55470

Files:
  lib/Format/ContinuationIndenter.cpp
  unittests/Format/FormatTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55470.177374.patch
Type: text/x-patch
Size: 4340 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181208/19c26064/attachment.bin>