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

[Francois Ferrand via Phabricator via cfe-commits]

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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits