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

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 3 13:03:42 PDT 2019


MyDeveloperDay added a comment.

what if you have this

  vector<int>{1, 2, 3};
  new int[3]{1, 2, 3};
  foo(int f);

but want this:

  vector<int>{ 1, 2, 3 };
  new int[3]{ 1, 2, 3 };
  foo(int f);

wouldn't turning on  `SpacesInParentheses: true` now mean you get

  vector<int>{ 1, 2, 3 };
  new int[3]{ 1, 2, 3 };
  foo( int f );

so there is no way of still having `foo(int f);` without a space before the int and after the f?

Sorry did I miss something? Generally, I agree that the change is good for the braced case I just don't want to break everyones functions


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