[PATCH] D116283: [clang-format] Add an option to add a space between operator overloading and opening parentheses

Rajat Bajpai via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 31 05:12:55 PST 2021


rajatbajpai added inline comments.


================
Comment at: clang/unittests/Format/FormatTest.cpp:14545
   verifyFormat("auto lambda = []() { return 0; };", SomeSpace2);
+
+  FormatStyle SpaceAfterOperatorOverloading = getLLVMStyle();
----------------
HazardyKnusperkeks wrote:
> rajatbajpai wrote:
> > MyDeveloperDay wrote:
> > > There should be a PARSE unit test too please
> > I'm sorry if I misunderstood it, do you mean tests using CHECK_PARSE? I am confirming this because I didn't find any such test case for existing //SpaceBeforeParensOptions// options.
> Than this has slipped through, but there should be a test for every parsing aspect.
Thanks for confirming. 

Not an issue, I'll try to add a test case for existing options as well then.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116283



More information about the llvm-commits mailing list