[PATCH] D30487: ClangFormat - Add option to break before inheritance separation operator in class declaration

Daniel Jasper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 3 05:09:17 PST 2017


djasper added a comment.

Hm. Unfortunately, this seems to have been implemented to support Webkit style and Webkit style is explicit about wanting this (https://webkit.org/code-style-guidelines/) :(.

But maybe the solution to that is to add an extra flag like AlwaysWrapInitializerList. Really not sure how best to organize this. Any thoughts? (I personally care about neither of these styles, so maybe I am not the best to judge)

At any rate, to move forward, could you remove the hasMultipleInheritance function and instead use the alternative approach discussed?


Repository:
  rL LLVM

https://reviews.llvm.org/D30487





More information about the cfe-commits mailing list