[PATCH] add option to break constructor initializers before colon

Daniel Jasper djasper at google.com
Sun Dec 8 12:32:09 PST 2013


I think we might just want to change the existing
option BreakConstructorInitializersBeforeComma to do exactly this. WebKit
style is the only style using it and I agree that the style guide is
explicit in this regard.


On Thu, Dec 5, 2013 at 10:50 PM, Nico Weber <thakis at chromium.org> wrote:

> (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
>>
>>
>
> _______________________________________________
> 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/20131208/a2c1a077/attachment.html>


More information about the cfe-commits mailing list