[PATCH] D110833: [clang-format] Refactor SpaceBeforeParens to add options

Christian Rayroud via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 3 00:19:36 PDT 2021


crayroud marked an inline comment as done.
crayroud added inline comments.


================
Comment at: clang/unittests/Format/FormatTest.cpp:14275
+  verifyFormat("A::A() : a (1) {}", SomeSpace2);
+  verifyFormat("void f() __attribute__ ((asdf));", SomeSpace2);
+  verifyFormat("*(&a + 1);\n"
----------------
HazardyKnusperkeks wrote:
> Is this really desired?
It is allowed to have a space after `__attribute__` and the behaviour is the same with `SpaceBeforeParens: NonEmptyParentheses`.


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

https://reviews.llvm.org/D110833



More information about the cfe-commits mailing list