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

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 30 12:11:42 PST 2021


HazardyKnusperkeks added inline comments.


================
Comment at: clang/unittests/Format/FormatTest.cpp:14545
   verifyFormat("auto lambda = []() { return 0; };", SomeSpace2);
+
+  FormatStyle SpaceAfterOperatorOverloading = getLLVMStyle();
----------------
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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116283



More information about the cfe-commits mailing list