[PATCH] D115879: [clang-format] extern with new line brace without indentation
    Owen Pan via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Fri Dec 17 00:31:19 PST 2021
    
    
  
owenpan added inline comments.
================
Comment at: clang/unittests/Format/FormatTest.cpp:3816
+  Style.IndentExternBlock = FormatStyle::IEBS_AfterExternBlock;
+  Style.BreakBeforeBraces = FormatStyle::BS_Custom;
+  Style.BraceWrapping.AfterExternBlock = true;
----------------
This line is unnecessary too.
================
Comment at: clang/unittests/Format/FormatTest.cpp:3826-3827
+  Style.IndentExternBlock = FormatStyle::IEBS_AfterExternBlock;
+  Style.BreakBeforeBraces = FormatStyle::BS_Custom;
+  Style.BraceWrapping.AfterExternBlock = true;
+  Style.IndentExternBlock = FormatStyle::IEBS_Indent;
----------------
I would remove these too.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D115879/new/
https://reviews.llvm.org/D115879
    
    
More information about the cfe-commits
mailing list