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

Francois Ferrand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 26 07:01:18 PDT 2017


Typz added a comment.

In https://reviews.llvm.org/D33447#765610, @djasper wrote:

> I think it's just wrong that WebKit inherits this. The style guide explicitly says that this is wrong:
>
>   MyOtherClass::MyOtherClass() : MySuperClass() {}


I think this exemple applies to constructors only.
Looking at webkit code, there are many shortl functions which are indeed formatted on a single line (at least 22097 occurences, actually)

> 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