[PATCH] D120217: [clang-format] Add an option to insert braces after control statements

Owen Pan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 21 01:38:27 PST 2022


owenpan marked an inline comment as done.
owenpan added inline comments.


================
Comment at: clang/lib/Format/Format.cpp:1850
+      for (FormatToken *Token = Line->First; Token; Token = Token->Next) {
+        if (Token->Finalized || Token->BraceCount == 0)
+          continue;
----------------
curdeius wrote:
> I think that we can skip the whole line if the first token on line is finalized.
Good point!


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

https://reviews.llvm.org/D120217



More information about the cfe-commits mailing list