[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