[PATCH] D68415: [clang-format] C++11 braced lists should respect the SpacesInParentheses setting

Mitchell via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 3 12:46:00 PDT 2019


mitchell-stellar added a comment.

I don't think that's a good idea, considering the fact that braces can mean different things in different contexts, and it would cause trouble for existing clang-format settings.

If a hypothetical `SpacesInBraces` were `false` by default, then you could have clang-format output something like:

`[](int foo) {bar(foo);}`

Currently, it is `[](int foo) { bar(foo); }`.

If a hypothetical `SpacesInBraces` were `true` by default, then you could have clang-format output something like:

`vector<int> x{ 1, 2, 3, 4 };`

Currently, it is `vector<int> x{1, 2, 3, 4};`.

This patch is minimally invasive and adheres to the existing documentation.


Repository:
  rC Clang

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

https://reviews.llvm.org/D68415





More information about the cfe-commits mailing list