[PATCH] D32479: [clang-format] Add BreakConstructorInitializersBeforeColon option

Francois Ferrand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 16 07:03:10 PDT 2017


Typz added a comment.

Or would it be better to replace (i.e. deprecate) the BreakConstructorInitializersBeforeComma option with a multiple choice option, as suggested here:
http://clang-developers.42468.n3.nabble.com/clang-format-Proposal-for-a-new-style-for-constructor-and-initializers-line-break-td4041595.html

  /// \brief Different ways to break initializers.
  enum BreakConstructorInitializersStyle
  {
    /// Constructor()
    ///     : initializer1(),
    ///       initializer2()
    BCIS_BeforeColonAfterComma,
    /// Constructor()
    ///     : initializer1()
    ///     , initializer2()
    BCIS_BeforeColonAndComma,
    /// Constructor() :
    ///     initializer1(),
    ///     initializer2()
    BCIS_AfterColonAndComma
  };
  BreakConstructorInitializersStyle BreakConstructorInitializers


https://reviews.llvm.org/D32479





More information about the cfe-commits mailing list