[PATCH] add option to break constructor initializers before colon

Nico Weber thakis at chromium.org
Thu Dec 5 13:50:36 PST 2013


(This is what WebKit style asks for too:
http://www.webkit.org/coding/coding-style.html , "Other Punctuation" point
1. So if this goes in, maybe you can set to true for webkit style.)


On Thu, Dec 5, 2013 at 10:23 AM, Florian Sowade <f.sowade at r9e.de> wrote:

> Hi,
>
> the attached patch adds a clang-format option to always break
> constructor initializers before the initial colon.
>
> So this patch adds an option to turn
>
> Constructor : foo(bar)
> {}
>
> into
>
> Constructor
>     : foo(bar)
> {}
>
> I don't know if this style is widely distributed, but I use it to
> minimize the diff when I add new members to the list of constructor
> initializers, since I can add a second member to the list without
> modifying any of the existing lines:
>
> Constructor
>     : foo(bar)
>     , baz()
> {}
>
> From my understanding this is not possible with the current set of
> options.
>
> Florian Sowade
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131205/916f3eea/attachment.html>


More information about the cfe-commits mailing list