<div dir="ltr">Not entirely sure what advise you are looking for.<div>My thoughts:<div>- I am happy to accept a patch provided is has sufficient unit tests</div><div>- I think this should go in as a single patch, I don't see a need for "a set of small patches"</div><div>- Personally, I think this style is bad as it makes it harder to distinguish wrapped constructor arguments from initializers</div><div>- In particular, how do you indent:</div><div><br></div><div>Constructor( // Arguments too long to fit on the same line</div><div>    SomeArgument some_argument,</div><div>    SomeOtherArgument some_other_argument) :</div><div>    some_argument_(some_argument),</div><div>    some_other_argument_(some_other_argument) {</div><div>}</div><div><br></div><div>Is this style used in any large project or widely used style guide?</div><div><br></div><div>Cheers,<br>Daniel</div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Sep 14, 2014 at 10:34 AM, Jean-philippe Dufraigne <span dir="ltr"><<a href="mailto:j.dufraigne@gmail.com" target="_blank">j.dufraigne@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hello,<br><br></div><div>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></div><div><br>BreakConstructorInitializers would replace 'bool BreakConstructorInitializersBeforeComma;' while maintaining backward compatibility, in a similar fashion it was done before.<br><br></div><div>I have the prototype for the code and would be really happy to finalize it and contribute a set of small patches.<br><br></div><div>Please let me know if this something that fits with the project and if you have any advice.<br></div><div>Best Regards,<br></div><div>Jean-Philippe<br></div><div><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></div></div>
<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" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
<br></blockquote></div><br></div>