[PATCH] D67773: [clang-format[PR43144] Add support for SpaceAroundBraces style

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 4 09:31:29 PDT 2019


MyDeveloperDay added a comment.

In D67773#1694797 <https://reviews.llvm.org/D67773#1694797>, @mitchell-stellar wrote:

> This needs to be more thought out. At the very least it needs to cover more keywords like `do`, `switch`, `try`, and `catch`. There may be more. Consideration also needs to be made for `class`, functions, `namespace`, etc. Otherwise this is an experimental feature at best. There is also confusion with how it may interact with `SpaceBeforeCpp11BracedList`. The name `SpacesAroundBraces` is just too generic in that regard considering the fact that braces play multiple roles in C++.


Agreed


Repository:
  rC Clang

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

https://reviews.llvm.org/D67773





More information about the cfe-commits mailing list