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



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

Reply via email to