From b362f6258eabe56443956024c2df7d0f785f3e4d Mon Sep 17 00:00:00 2001 From: Daniel Jasper Date: Thu, 22 May 2014 13:53:55 +0000 Subject: [PATCH] clang-format: Don't use Allman brace breaking for ObjC blocks. It just seems wrong. This fixes llvm.org/PR19736. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@209440 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Format/TokenAnnotator.cpp | 3 ++- unittests/Format/FormatTest.cpp | 10 ++++++++++ 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/lib/Format/TokenAnnotator.cpp b/lib/Format/TokenAnnotator.cpp index ce847d6427..f057ef7f3e 100644 --- a/lib/Format/TokenAnnotator.cpp +++ b/lib/Format/TokenAnnotator.cpp @@ -1595,7 +1595,8 @@ bool TokenAnnotator::mustBreakBefore(const AnnotatedLine &Line, Style.BreakConstructorInitializersBeforeComma && !Style.ConstructorInitializerAllOnOneLineOrOnePerLine) { return true; - } else if (Right.is(tok::l_brace) && (Right.BlockKind == BK_Block)) { + } else if (Right.is(tok::l_brace) && Right.BlockKind == BK_Block && + Right.Type != TT_ObjCBlockLBrace) { return Style.BreakBeforeBraces == FormatStyle::BS_Allman || Style.BreakBeforeBraces == FormatStyle::BS_GNU; } else if (Right.is(tok::string_literal) && diff --git a/unittests/Format/FormatTest.cpp b/unittests/Format/FormatTest.cpp index b3fb776a7a..9cf4efcbf1 100644 --- a/unittests/Format/FormatTest.cpp +++ b/unittests/Format/FormatTest.cpp @@ -7698,6 +7698,16 @@ TEST_F(FormatTest, AllmanBraceBreaking) { "#endif", BreakBeforeBrace); + // This shouldn't affect ObjC blocks. + verifyFormat("[self doSomeThingWithACompletionHandler:^{\n" + " // ...\n" + " int i;\n" + "}];"); + verifyFormat("void (^block)(void) = ^{\n" + " // ...\n" + " int i;\n" + "};"); + BreakBeforeBrace.ColumnLimit = 19; verifyFormat("void f() { int i; }", BreakBeforeBrace); BreakBeforeBrace.ColumnLimit = 18; -- 2.40.0