[PATCH] D109557: Adds a BreakBeforeClosingParen option

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 16 04:17:40 PDT 2021


MyDeveloperDay added inline comments.


================
Comment at: clang/unittests/Format/FormatTest.cpp:22410
+               ");",
+               Style);
+}
----------------
Can you assert the cases it won't do (given your description it seem like it would do it for all before ) cases)

```
int a = (int
) b;
```

I'm pretty sure it won't but lets me explicit in the tests

```
return (true
);
```

```
void foo(
);
```

etc..



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109557



More information about the cfe-commits mailing list