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

Mitchell via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 4 08:43:09 PDT 2019


mitchell-stellar requested changes to this revision.
mitchell-stellar added a comment.
This revision now requires changes to proceed.

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++.


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