[PATCH] D133571: [clang-format] Introduce NoFallThrough option into AllowShortCaseLabelsOnASingleLine

[Marek Kurdej via Phabricator via cfe-commits]

curdeius accepted this revision.
curdeius added a comment.
This revision is now accepted and ready to land.

LGTM after fixing the last comment.



================
Comment at: clang/lib/Format/UnwrappedLineFormatter.cpp:650
+      // Don't merge if the last thing on the line before was an attribute.
+      if (PreviousLine->Last->Previous && PreviousLine->Last->Previous) {
+        auto *PrevPrev = PreviousLine->Last->Previous->Previous;
----------------
Haven't seen this before.


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

https://reviews.llvm.org/D133571