[PATCH] D55170: [clang-format]: Add NonEmptyParentheses spacing option

Mark Zeren via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 20 06:38:02 PDT 2019


mzeren-vmw added a comment.

In D55170#1436087 <https://reviews.llvm.org/D55170#1436087>, @klimek wrote:

> Oh wow, wasn't aware how popular this is. Sigh. I'd like us to push them to change their style guide, but I guess that's not going to fly.


IDK. If the criteria is "public coding standard for widely used project", supporting all such things may be unsustainable. Some projects may need to fork or find another formatter. I speak as someone that maintains an internal fork with features that I do not deem worthy of foisting on trunk.
(And BTW @MyDeveloperDay, whoever you are, Thanks for all of your recent activity!)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55170/new/

https://reviews.llvm.org/D55170





More information about the cfe-commits mailing list