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

Fri Mar 3 05:35:23 PST 2017

Abpostelnicu added a comment.

In https://reviews.llvm.org/D30487#691535, @djasper wrote:

> 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?

Sure will drop hasMultipleInheritance and use the variable that you mentioned and i won't touch the rest of the patch. After that i will repost it here.

Repository:
  rL LLVM

https://reviews.llvm.org/D30487