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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to