[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