[PATCH] D114583: [clang-format] Adjust braced list detection

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 13 03:59:23 PST 2021


MyDeveloperDay added a comment.

We might want to revert this until we get a solution to avoid flip flopping peoples formats, but this to me comes down to what we've been saying before about using the TT_ types to labels the types of `() {} []`

I think in hindsight adding l_brace here just too generic (we need to know if its a "BracedInitializer {" or just a scope "{"

  (NextTok->isOneOf(tok::l_brace, tok::identifier)




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114583



More information about the cfe-commits mailing list