diff options
Diffstat (limited to 'clang/unittests/Format/FormatTest.cpp')
| -rw-r--r-- | clang/unittests/Format/FormatTest.cpp | 36 | 
1 files changed, 20 insertions, 16 deletions
| diff --git a/clang/unittests/Format/FormatTest.cpp b/clang/unittests/Format/FormatTest.cpp index d45babe..ca9e792 100644 --- a/clang/unittests/Format/FormatTest.cpp +++ b/clang/unittests/Format/FormatTest.cpp @@ -5126,7 +5126,8 @@ TEST_F(FormatTest, DesignatedInitializers) {  TEST_F(FormatTest, BracedInitializerIndentWidth) {    auto Style = getLLVMStyleWithColumns(60);    Style.BinPackArguments = true; -  Style.AlignAfterOpenBracket = FormatStyle::BAS_AlwaysBreak; +  Style.BreakAfterOpenBracketFunction = true; +  Style.BreakAfterOpenBracketBracedList = true;    Style.BracedInitializerIndentWidth = 6;    // Non-initializing braces are unaffected by BracedInitializerIndentWidth. @@ -5302,7 +5303,8 @@ TEST_F(FormatTest, BracedInitializerIndentWidth) {                 Style);    // Aligning after open braces unaffected by BracedInitializerIndentWidth. -  Style.AlignAfterOpenBracket = FormatStyle::BAS_Align; +  Style.AlignAfterOpenBracket = true; +  Style.BreakAfterOpenBracketBracedList = false;    verifyFormat("SomeStruct s{\"xxxxxxxxxxxxx\", \"yyyyyyyyyyyyy\",\n"                 "             \"zzzzzzzzzzzzz\"};",                 Style); @@ -7459,7 +7461,7 @@ TEST_F(FormatTest, ExpressionIndentationBreakingBeforeOperators) {    Style.IndentWidth = 4;    Style.TabWidth = 4;    Style.UseTab = FormatStyle::UT_Always; -  Style.AlignAfterOpenBracket = FormatStyle::BAS_DontAlign; +  Style.AlignAfterOpenBracket = false;    Style.AlignOperands = FormatStyle::OAS_DontAlign;    verifyFormat("return someVeryVeryLongConditionThatBarelyFitsOnALine\n"                 "\t&& (someOtherLongishConditionPart1\n" @@ -7470,7 +7472,7 @@ TEST_F(FormatTest, ExpressionIndentationBreakingBeforeOperators) {                 Style);    Style = getLLVMStyleWithColumns(20); -  Style.AlignAfterOpenBracket = FormatStyle::BAS_AlwaysBreak; +  Style.BreakAfterOpenBracketFunction = true;    Style.BinPackParameters = FormatStyle::BPPS_OnePerLine;    Style.BreakBeforeBinaryOperators = FormatStyle::BOS_NonAssignment;    Style.ContinuationIndentWidth = 2; @@ -7632,7 +7634,7 @@ TEST_F(FormatTest, NoOperandAlignment) {                 "        * cccccccccccccccccccccccccccccccccccc;",                 Style); -  Style.AlignAfterOpenBracket = FormatStyle::BAS_DontAlign; +  Style.AlignAfterOpenBracket = false;    verifyFormat("return (a > b\n"                 "    // comment1\n"                 "    // comment2\n" @@ -11248,7 +11250,7 @@ TEST_F(FormatTest, BreakBeforeTemplateCloser) {  TEST_F(FormatTest, WrapsTemplateParameters) {    FormatStyle Style = getLLVMStyle(); -  Style.AlignAfterOpenBracket = FormatStyle::BAS_DontAlign; +  Style.AlignAfterOpenBracket = false;    Style.BreakBeforeBinaryOperators = FormatStyle::BOS_None;    verifyFormat(        "template <typename... a> struct q {};\n" @@ -11256,7 +11258,7 @@ TEST_F(FormatTest, WrapsTemplateParameters) {        "    aaaaaaaaaaaaaaaaa, aaaaaaaaaaaaaaaaa, aaaaaaaaaaaaaaaaa>\n"        "    y;",        Style); -  Style.AlignAfterOpenBracket = FormatStyle::BAS_DontAlign; +  Style.AlignAfterOpenBracket = false;    Style.BreakBeforeBinaryOperators = FormatStyle::BOS_All;    verifyFormat(        "template <typename... a> struct r {};\n" @@ -11264,7 +11266,7 @@ TEST_F(FormatTest, WrapsTemplateParameters) {        "    aaaaaaaaaaaaaaaaa, aaaaaaaaaaaaaaaaa, aaaaaaaaaaaaaaaaa>\n"        "    y;",        Style); -  Style.AlignAfterOpenBracket = FormatStyle::BAS_AlwaysBreak; +  Style.BreakAfterOpenBracketFunction = true;    Style.BreakBeforeBinaryOperators = FormatStyle::BOS_None;    verifyFormat("template <typename... a> struct s {};\n"                 "extern s<\n" @@ -11274,7 +11276,7 @@ TEST_F(FormatTest, WrapsTemplateParameters) {                 "aaaaaaaaaaaaaaaaaaaaaa>\n"                 "    y;",                 Style); -  Style.AlignAfterOpenBracket = FormatStyle::BAS_AlwaysBreak; +  Style.BreakAfterOpenBracketFunction = true;    Style.BreakBeforeBinaryOperators = FormatStyle::BOS_All;    verifyFormat("template <typename... a> struct t {};\n"                 "extern t<\n" @@ -14302,7 +14304,7 @@ TEST_F(FormatTest, LayoutCxx11BraceInitializers) {                 "};",                 NoBinPacking); -  NoBinPacking.AlignAfterOpenBracket = FormatStyle::BAS_AlwaysBreak; +  NoBinPacking.BreakAfterOpenBracketBracedList = true;    verifyFormat("static uint8 CddDp83848Reg[] = {\n"                 "    CDDDP83848_BMCR_REGISTER,\n"                 "    CDDDP83848_BMSR_REGISTER,\n" @@ -15972,13 +15974,14 @@ TEST_F(FormatTest, BreaksStringLiteralOperands) {    // In a function call with two operands, with AlignAfterOpenBracket enabled,    // the first must be broken with a line break before it.    FormatStyle Style = getLLVMStyleWithColumns(25); -  Style.AlignAfterOpenBracket = FormatStyle::BAS_AlwaysBreak; +  Style.BreakAfterOpenBracketFunction = true;    verifyFormat("someFunction(\n"                 "    \"long long long \"\n"                 "    \"long\",\n"                 "    a);",                 "someFunction(\"long long long long\", a);", Style); -  Style.AlignAfterOpenBracket = FormatStyle::BAS_BlockIndent; +  Style.BreakAfterOpenBracketFunction = true; +  Style.BreakBeforeCloseBracketFunction = true;    verifyFormat("someFunction(\n"                 "    \"long long long \"\n"                 "    \"long\",\n" @@ -17773,7 +17776,7 @@ TEST_F(FormatTest, ConfigurableSpacesInParens) {    Spaces.ColumnLimit = 80;    Spaces.IndentWidth = 4; -  Spaces.AlignAfterOpenBracket = FormatStyle::BAS_AlwaysBreak; +  Spaces.BreakAfterOpenBracketFunction = true;    verifyFormat("void foo( ) {\n"                 "    size_t foo = (*(function))(\n"                 "        Foooo, Barrrrr, Foooo, Barrrr, FoooooooooLooooong, " @@ -17798,7 +17801,8 @@ TEST_F(FormatTest, ConfigurableSpacesInParens) {                 "}",                 Spaces); -  Spaces.AlignAfterOpenBracket = FormatStyle::BAS_BlockIndent; +  Spaces.BreakAfterOpenBracketFunction = true; +  Spaces.BreakBeforeCloseBracketFunction = true;    verifyFormat("void foo( ) {\n"                 "    size_t foo = (*(function))(\n"                 "        Foooo, Barrrrr, Foooo, Barrrr, FoooooooooLooooong, " @@ -22827,7 +22831,7 @@ TEST_F(FormatTest, ConstructorInitializerIndentWidth) {        ": aaaaaaaaaaaaa(aaaaaaaaaaaaaa), aaaaaaaaaaaaa(aaaaaaaaaaaaaa),\n"        "  aaaaaaaaaaaaa(aaaaaaaaaaaaaa) {}",        Style); -  Style.AlignAfterOpenBracket = FormatStyle::BAS_AlwaysBreak; +  Style.BreakAfterOpenBracketFunction = true;    verifyFormat(        "SomeLongTemplateVariableName<\n"        "    aaaaaaaaaaaaaaaaaaaaaaaaaaaaaa, aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa>", @@ -24082,7 +24086,7 @@ TEST_F(FormatTest, FormatsLambdas) {                 "      return aFunkyFunctionCall(qux);\n"                 "    }} {}",                 Style); -  Style.AlignAfterOpenBracket = FormatStyle::BAS_AlwaysBreak; +  Style.BreakAfterOpenBracketFunction = true;    // FIXME: The following test should pass, but fails at the time of writing.  #if 0    // As long as all the non-lambda arguments fit on a single line, AlwaysBreak | 
