[PATCH] D150614: [clang-format] Ignore first token when finding MustBreak

Emilia Kond via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 15 23:35:12 PDT 2023


rymiel added inline comments.


================
Comment at: clang/lib/Format/UnwrappedLineFormatter.cpp:891-895
+    for (const FormatToken *Tok = Line->First; Tok; Tok = Tok->Next) {
+      // Ignore the first token, because in this situation, it applies more
+      // to the last token of the previous line.
+      if (Tok == Line->First)
+        continue;
----------------
owenpan wrote:
> We can start from the second token.
Thanks! I'm really glad to have reviewers to catch times where I wasn't thinking straight


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150614



More information about the cfe-commits mailing list