[PATCH] D94500: Rework Whitesmiths mode to use line-level values in UnwrappedLineParser

Marek Kurdej via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 27 12:30:28 PST 2021


curdeius added inline comments.


================
Comment at: clang/unittests/Format/FormatTest.cpp:13696
                "      }\n"
-               "    break;\n"
+               "      break;\n"
                "    }\n"
----------------
curdeius wrote:
> MyDeveloperDay wrote:
> > whilst I don't really like changing the indentation I don't think there was anything that says it was correct before.
> That is really strange that `case` is indented differently than `break`. Is it intended? Is the style really like this?
Forget it. I was thinking about something else.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94500



More information about the cfe-commits mailing list