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

Manuel Klimek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 20 03:11:15 PDT 2019


klimek added a comment.

In D55170#1432832 <https://reviews.llvm.org/D55170#1432832>, @reuk wrote:

> @klimek I agree that the rule is somewhat arbitrary. However, it's the style rule of an established codebase with many users (I don't have a concrete number, but the project has 1400 stars on github <https://github.com/weareroli/juce>). I've found this patch useful when contributing to JUCE and I thought others might too.


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.


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

https://reviews.llvm.org/D55170





More information about the cfe-commits mailing list