[PATCH] D82016: [clang-format] [PR462254] fix indentation of default and break correctly in whitesmiths style
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jun 17 08:03:50 PDT 2020
MyDeveloperDay created this revision.
MyDeveloperDay added reviewers: krasimir, JakeMerdichAMD, curdeius, jbcoe, timwoj.
MyDeveloperDay added projects: clang, clang-format.
https://bugs.llvm.org/show_bug.cgi?id=46254
In ‘clang-format’, when using "BreakBeforeBraces: Whitesmiths".
In a ‘switch’ block, the ‘default’ case is always missing and indentation level. See below.
switch(x)
{
case 0:
{
foo(x + 1);
}
break;
case 1:
{
goo(x % 42);
}
break;
default:
{
printf("the default case");
}
break;
}
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D82016
Files:
clang/lib/Format/UnwrappedLineFormatter.cpp
clang/lib/Format/UnwrappedLineParser.cpp
clang/unittests/Format/FormatTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D82016.271373.patch
Type: text/x-patch
Size: 5026 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200617/6d7b5a22/attachment.bin>
More information about the cfe-commits
mailing list