<div dir="ltr">Looks good. Thanks for the patch!<div><br>I've fixed a couple of nits and committed it as r197386.<br><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Dec 16, 2013 at 11:46 AM, Florian Sowade <span dir="ltr"><<a href="mailto:f.sowade@r9e.de" target="_blank">f.sowade@r9e.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi,<br>
<div class="im"><br>
On Sun, 2013-12-08 at 21:32 +0100, Daniel Jasper wrote:<br>
> I think we might just want to change the existing<br>
> option BreakConstructorInitializersBeforeComma to do exactly this. WebKit<br>
> style is the only style using it and I agree that the style guide is<br>
> explicit in this regard.<br>
<br>
</div>that seems reasonable to me. Please find attached a patch which<br>
implements that behavior.<br>
<br>
Unlike the old one, the new patch also works if ColumnLimit is 0, so it<br>
does work for WebKit, too.<br>
<span class=""><font color="#888888"><br>
Florian Sowade<br>
</font></span><div class=""><div class="h5"><br>
> On Thu, Dec 5, 2013 at 10:50 PM, Nico Weber <<a href="mailto:thakis@chromium.org">thakis@chromium.org</a>> wrote:<br>
><br>
> > (This is what WebKit style asks for too:<br>
> > <a href="http://www.webkit.org/coding/coding-style.html" target="_blank">http://www.webkit.org/coding/coding-style.html</a> , "Other Punctuation"<br>
> > point 1. So if this goes in, maybe you can set to true for webkit style.)<br>
> ><br>
> ><br>
> > On Thu, Dec 5, 2013 at 10:23 AM, Florian Sowade <<a href="mailto:f.sowade@r9e.de">f.sowade@r9e.de</a>> wrote:<br>
> ><br>
> >> Hi,<br>
> >><br>
> >> the attached patch adds a clang-format option to always break<br>
> >> constructor initializers before the initial colon.<br>
> >><br>
> >> So this patch adds an option to turn<br>
> >><br>
> >> Constructor : foo(bar)<br>
> >> {}<br>
> >><br>
> >> into<br>
> >><br>
> >> Constructor<br>
> >>     : foo(bar)<br>
> >> {}<br>
> >><br>
> >> I don't know if this style is widely distributed, but I use it to<br>
> >> minimize the diff when I add new members to the list of constructor<br>
> >> initializers, since I can add a second member to the list without<br>
> >> modifying any of the existing lines:<br>
> >><br>
> >> Constructor<br>
> >>     : foo(bar)<br>
> >>     , baz()<br>
> >> {}<br>
> >><br>
> >> From my understanding this is not possible with the current set of<br>
> >> options.<br>
> >><br>
> >> Florian Sowade</div></div></blockquote></div>
</div></div></div></div>