<p dir="ltr">On 14 Sep 2014 02:35, "Daniel Jasper" <<a href="mailto:djasper@google.com">djasper@google.com</a>> wrote:<br>
><br>
> Not entirely sure what advise you are looking for.<br>
> My thoughts:<br>
> - I am happy to accept a patch provided is has sufficient unit tests<br>
> - I think this should go in as a single patch, I don't see a need for "a set of small patches"<br>
> - Personally, I think this style is bad as it makes it harder to distinguish wrapped constructor arguments from initializers<br>
> - In particular, how do you indent:<br>
><br>
> Constructor( // Arguments too long to fit on the same line<br>
>     SomeArgument some_argument,<br>
>     SomeOtherArgument some_other_argument) :<br>
>     some_argument_(some_argument),<br>
>     some_other_argument_(some_other_argument) {<br>
> }<br>
><br>
> Is this style used in any large project or widely used style guide?</p>
<p dir="ltr">I've worked on a large (closed source) project that used this style. These lists got line continuation indent (4 space) rather than scope indent (2 space) on that project.</p>
<p dir="ltr">> Cheers,<br>
> Daniel<br>
><br>
><br>
> On Sun, Sep 14, 2014 at 10:34 AM, Jean-philippe Dufraigne <<a href="mailto:j.dufraigne@gmail.com">j.dufraigne@gmail.com</a>> wrote:<br>
>><br>
>> Hello,<br>
>><br>
>> I'm interested in adding support for another style for constructor colon and initializer.<br>
>> An example of the style is above the value BCIS_AfterColonAndComma bellow.<br>
>><br>
>> BreakConstructorInitializers would replace 'bool BreakConstructorInitializersBeforeComma;' while maintaining backward compatibility, in a similar fashion it was done before.<br>
>><br>
>> I have the prototype for the code and would be really happy to finalize it and contribute a set of small patches.<br>
>><br>
>> Please let me know if this something that fits with the project and if you have any advice.<br>
>> Best Regards,<br>
>> Jean-Philippe<br>
>><br>
>>   /// \brief Different ways to break initializers.<br>
>>   enum BreakConstructorInitializersStyle<br>
>>   {<br>
>>     /// Constructor()<br>
>>     ///     : initializer1(),<br>
>>     ///       initializer2()<br>
>>     BCIS_BeforeColonAfterComma,<br>
>>     /// Constructor()<br>
>>     ///     : initializer1()<br>
>>     ///     , initializer2()<br>
>>     BCIS_BeforeColonAndComma,<br>
>>     /// Constructor() :<br>
>>     ///     initializer1(),<br>
>>     ///     initializer2()<br>
>>     BCIS_AfterColonAndComma<br>
>>   };<br>
>><br>
>>   /// \brief Breaking and alignment style for constructor initializers<br>
>>   BreakConstructorInitializersStyle BreakConstructorInitializers;<br>
>><br>
>><br>
>> _______________________________________________<br>
>> cfe-dev mailing list<br>
>> <a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
>><br>
><br>
><br>
> _______________________________________________<br>
> cfe-dev mailing list<br>
> <a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
><br>
</p>