[PATCH] D33447: clang-format: add option to merge empty function body

Daniel Jasper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 26 06:26:26 PDT 2017


djasper added a comment.

I think it's just wrong that WebKit inherits this. The style guide explicitly says that this is wrong:

  MyOtherClass::MyOtherClass() : MySuperClass() {}

So I still think we can make this work with the existing options without regressing a behavior that anyone is relying on.


https://reviews.llvm.org/D33447





More information about the cfe-commits mailing list