[PATCH] D145344: [clang-format] Don't annotate left brace of class as FunctionLBrace

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 7 02:38:41 PST 2023


MyDeveloperDay added a comment.

The upshot of this is better TT_FunctionLBrace/TT_ClassLBrace detection, which is great becuase  if a class `{` or function `{` is incorrectly detected we are then in unknown territory as to how it might become formatted, since you introduced these Annotator tests, I fell we are really helping to lock down the other rules.  I would hazard a guess that this actually solves more issues than we realize.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145344



More information about the cfe-commits mailing list