[PATCH] D121450: [clang-format] Handle attributes before case label.

Marek Kurdej via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 22 06:45:33 PDT 2022


curdeius added inline comments.


================
Comment at: clang/lib/Format/UnwrappedLineParser.cpp:1826
+      if (Style.isJavaScript() && Line->MustBeDeclaration)
+        // 'case: string' field declaration.
+        break;
----------------
HazardyKnusperkeks wrote:
> Here's the loop. In the switch before that worked, since it was not in a loop. Here we need to do something. (nextToken()?)
Yes, it seems so. I'll add a test case and fix it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121450



More information about the cfe-commits mailing list