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

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 26 10:17:38 PST 2021


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

LGTM I think, it would be good to get input from other reviewers



================
Comment at: clang/unittests/Format/FormatTest.cpp:13696
                "      }\n"
-               "    break;\n"
+               "      break;\n"
                "    }\n"
----------------
whilst I don't really like changing the indentation I don't think there was anything that says it was correct before.


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