[PATCH] D114696: [clang-format] regressed default behavior for operator parentheses

C. Rayroud via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 29 08:38:54 PST 2021


crayroud added inline comments.


================
Comment at: clang/unittests/Format/FormatTest.cpp:14126-14127
+  // FIXME these tests regressed behaviour.
+  // verifyFormat("T A::operator() ();", Space);
+  // verifyFormat("X A::operator++ (T);", Space);
   verifyFormat("auto lambda = [] () { return 0; };", Space);
----------------
Why did you comment these lines? When SpaceBeforeParens is set to always it is not correct to have a space before the opening parenthesis, for operators ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114696



More information about the cfe-commits mailing list