Skip to content

Commit 9917b99

Browse files
author
Ivan Rymarchyk
committed
fixed PR comments
1 parent 9dbd674 commit 9917b99

File tree

9 files changed

+61
-131
lines changed

9 files changed

+61
-131
lines changed

‎.gitignore‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,4 +73,3 @@ pythonenv*
7373
/clang/utils/analyzer/projects/*/RefScanBuildResults
7474
# automodapi puts generated documentation files here.
7575
/lldb/docs/python_api/
76-
clang-build/

‎clang/include/clang/Format/Format.h‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -935,6 +935,18 @@ struct FormatStyle {
935935
ShortFunctionStyle(bool Empty, bool Inline, bool Other)
936936
: Empty(Empty), Inline(Inline), Other(Other) {}
937937
bool isAll() const { return Empty && Inline && Other; }
938+
static ShortFunctionStyle setEmptyOnly() {
939+
return ShortFunctionStyle(true, false, false);
940+
}
941+
static ShortFunctionStyle setEmptyAndInline() {
942+
return ShortFunctionStyle(true, true, false);
943+
}
944+
static ShortFunctionStyle setInlineOnly() {
945+
return ShortFunctionStyle(false, true, false);
946+
}
947+
static ShortFunctionStyle setAll() {
948+
return ShortFunctionStyle(true, true, true);
949+
}
938950
};
939951

940952
/// Dependent on the value, ``int f() { return 0; }`` can be put on a

‎clang/lib/Format/Format.cpp‎

Lines changed: 11 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -626,27 +626,17 @@ template <> struct MappingTraits<FormatStyle::ShortFunctionStyle> {
626626
static void enumInput(IO &IO, FormatStyle::ShortFunctionStyle &Value) {
627627
IO.enumCase(Value, "None", FormatStyle::ShortFunctionStyle({}));
628628
IO.enumCase(Value, "Empty",
629-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
630-
/*Inline=*/false,
631-
/*Other=*/false}));
629+
FormatStyle::ShortFunctionStyle::setEmptyOnly());
632630
IO.enumCase(Value, "Inline",
633631
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
634632
/*Inline=*/true,
635633
/*Other=*/false}));
636634
IO.enumCase(Value, "InlineOnly",
637-
FormatStyle::ShortFunctionStyle({/*Empty=*/false,
638-
/*Inline=*/true,
639-
/*Other=*/false}));
640-
IO.enumCase(Value, "All",
641-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
642-
/*Inline=*/true,
643-
/*Other=*/true}));
635+
FormatStyle::ShortFunctionStyle::setInlineOnly());
636+
IO.enumCase(Value, "All", FormatStyle::ShortFunctionStyle::setAll());
644637

645638
// For backward compatibility.
646-
IO.enumCase(Value, "true",
647-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
648-
/*Inline=*/true,
649-
/*Other=*/true}));
639+
IO.enumCase(Value, "true", FormatStyle::ShortFunctionStyle::setAll());
650640
IO.enumCase(Value, "false", FormatStyle::ShortFunctionStyle({}));
651641
}
652642

@@ -1534,9 +1524,7 @@ FormatStyle getLLVMStyle(FormatStyle::LanguageKind Language) {
15341524
LLVMStyle.AllowShortCompoundRequirementOnASingleLine = true;
15351525
LLVMStyle.AllowShortEnumsOnASingleLine = true;
15361526
LLVMStyle.AllowShortFunctionsOnASingleLine =
1537-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
1538-
/*Inline=*/true,
1539-
/*Other=*/true});
1527+
FormatStyle::ShortFunctionStyle::setAll();
15401528
LLVMStyle.AllowShortIfStatementsOnASingleLine = FormatStyle::SIS_Never;
15411529
LLVMStyle.AllowShortLambdasOnASingleLine = FormatStyle::SLS_All;
15421530
LLVMStyle.AllowShortLoopsOnASingleLine = false;
@@ -1825,9 +1813,7 @@ FormatStyle getGoogleStyle(FormatStyle::LanguageKind Language) {
18251813
GoogleStyle.AlignTrailingComments = {};
18261814
GoogleStyle.AlignTrailingComments.Kind = FormatStyle::TCAS_Never;
18271815
GoogleStyle.AllowShortFunctionsOnASingleLine =
1828-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
1829-
/*Inline=*/false,
1830-
/*Other=*/false});
1816+
FormatStyle::ShortFunctionStyle::setEmptyOnly();
18311817
GoogleStyle.AllowShortIfStatementsOnASingleLine = FormatStyle::SIS_Never;
18321818
GoogleStyle.AlwaysBreakBeforeMultilineStrings = false;
18331819
GoogleStyle.BreakBeforeBinaryOperators = FormatStyle::BOS_NonAssignment;
@@ -1838,9 +1824,7 @@ FormatStyle getGoogleStyle(FormatStyle::LanguageKind Language) {
18381824
GoogleStyle.AlignAfterOpenBracket = FormatStyle::BAS_AlwaysBreak;
18391825
GoogleStyle.AlignOperands = FormatStyle::OAS_DontAlign;
18401826
GoogleStyle.AllowShortFunctionsOnASingleLine =
1841-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
1842-
/*Inline=*/false,
1843-
/*Other=*/false});
1827+
FormatStyle::ShortFunctionStyle::setEmptyOnly();
18441828
// TODO: still under discussion whether to switch to SLS_All.
18451829
GoogleStyle.AllowShortLambdasOnASingleLine = FormatStyle::SLS_Empty;
18461830
GoogleStyle.AlwaysBreakBeforeMultilineStrings = false;
@@ -1858,9 +1842,7 @@ FormatStyle getGoogleStyle(FormatStyle::LanguageKind Language) {
18581842
GoogleStyle.SpacesInContainerLiterals = false;
18591843
} else if (Language == FormatStyle::LK_Proto) {
18601844
GoogleStyle.AllowShortFunctionsOnASingleLine =
1861-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
1862-
/*Inline=*/false,
1863-
/*Other=*/false});
1845+
FormatStyle::ShortFunctionStyle::setEmptyOnly();
18641846
GoogleStyle.AlwaysBreakBeforeMultilineStrings = false;
18651847
// This affects protocol buffer options specifications and text protos.
18661848
// Text protos are currently mostly formatted inside C++ raw string literals
@@ -1880,9 +1862,7 @@ FormatStyle getGoogleStyle(FormatStyle::LanguageKind Language) {
18801862
tooling::IncludeStyle::IBS_Preserve;
18811863
} else if (Language == FormatStyle::LK_CSharp) {
18821864
GoogleStyle.AllowShortFunctionsOnASingleLine =
1883-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
1884-
/*Inline=*/false,
1885-
/*Other=*/false});
1865+
FormatStyle::ShortFunctionStyle::setEmptyOnly();
18861866
GoogleStyle.AllowShortIfStatementsOnASingleLine = FormatStyle::SIS_Never;
18871867
GoogleStyle.BreakStringLiterals = false;
18881868
GoogleStyle.ColumnLimit = 100;
@@ -1942,9 +1922,7 @@ FormatStyle getChromiumStyle(FormatStyle::LanguageKind Language) {
19421922
} else {
19431923
ChromiumStyle.AllowAllParametersOfDeclarationOnNextLine = false;
19441924
ChromiumStyle.AllowShortFunctionsOnASingleLine =
1945-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
1946-
/*Inline=*/true,
1947-
/*Other=*/false});
1925+
FormatStyle::ShortFunctionStyle::setEmptyAndInline();
19481926
ChromiumStyle.AllowShortIfStatementsOnASingleLine = FormatStyle::SIS_Never;
19491927
ChromiumStyle.AllowShortLoopsOnASingleLine = false;
19501928
ChromiumStyle.BinPackParameters = FormatStyle::BPPS_OnePerLine;
@@ -1959,9 +1937,7 @@ FormatStyle getMozillaStyle() {
19591937
FormatStyle MozillaStyle = getLLVMStyle();
19601938
MozillaStyle.AllowAllParametersOfDeclarationOnNextLine = false;
19611939
MozillaStyle.AllowShortFunctionsOnASingleLine =
1962-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
1963-
/*Inline=*/true,
1964-
/*Other=*/false});
1940+
FormatStyle::ShortFunctionStyle::setEmptyAndInline();
19651941
MozillaStyle.AlwaysBreakAfterDefinitionReturnType =
19661942
FormatStyle::DRTBS_TopLevel;
19671943
MozillaStyle.BinPackArguments = false;

‎clang/lib/Format/UnwrappedLineFormatter.cpp‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -299,7 +299,8 @@ class LineJoiner {
299299
return true;
300300
}
301301

302-
if (Style.AllowShortFunctionsOnASingleLine.Inline) {
302+
if (Style.AllowShortFunctionsOnASingleLine.Inline &&
303+
!Style.AllowShortFunctionsOnASingleLine.Other) {
303304
// Just checking TheLine->Level != 0 is not enough, because it
304305
// provokes treating functions inside indented namespaces as short.
305306
if (Style.isJavaScript() && TheLine->Last->is(TT_FunctionLBrace))

‎clang/unittests/Format/ConfigParseTest.cpp‎

Lines changed: 5 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -624,34 +624,24 @@ TEST(ConfigParseTest, ParsesConfiguration) {
624624
FormatStyle::ShortFunctionStyle({}));
625625
CHECK_PARSE("AllowShortFunctionsOnASingleLine: Inline",
626626
AllowShortFunctionsOnASingleLine,
627-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
628-
/*Inline=*/true,
629-
/*Other=*/false}));
627+
FormatStyle::ShortFunctionStyle::setEmptyAndInline());
630628
CHECK_PARSE("AllowShortFunctionsOnASingleLine: Empty",
631629
AllowShortFunctionsOnASingleLine,
632-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
633-
/*Inline=*/false,
634-
/*Other=*/false}));
630+
FormatStyle::ShortFunctionStyle::setEmptyOnly());
635631
CHECK_PARSE("AllowShortFunctionsOnASingleLine: All",
636632
AllowShortFunctionsOnASingleLine,
637-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
638-
/*Inline=*/true,
639-
/*Other=*/true}));
633+
FormatStyle::ShortFunctionStyle::setAll());
640634
CHECK_PARSE("AllowShortFunctionsOnASingleLine: InlineOnly",
641635
AllowShortFunctionsOnASingleLine,
642-
FormatStyle::ShortFunctionStyle({/*Empty=*/false,
643-
/*Inline=*/true,
644-
/*Other=*/false}));
636+
FormatStyle::ShortFunctionStyle::setInlineOnly());
645637

646638
// For backward compatibility:
647639
CHECK_PARSE("AllowShortFunctionsOnASingleLine: false",
648640
AllowShortFunctionsOnASingleLine,
649641
FormatStyle::ShortFunctionStyle({}));
650642
CHECK_PARSE("AllowShortFunctionsOnASingleLine: true",
651643
AllowShortFunctionsOnASingleLine,
652-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
653-
/*Inline=*/true,
654-
/*Other=*/true}));
644+
FormatStyle::ShortFunctionStyle::setAll());
655645

656646
Style.AllowShortLambdasOnASingleLine = FormatStyle::SLS_All;
657647
CHECK_PARSE("AllowShortLambdasOnASingleLine: None",

‎clang/unittests/Format/FormatTest.cpp‎

Lines changed: 22 additions & 54 deletions
Original file line numberDiff line numberDiff line change
@@ -389,9 +389,7 @@ TEST_F(FormatTest, RemovesEmptyLines) {
389389

390390
FormatStyle Style = getLLVMStyle();
391391
Style.AllowShortFunctionsOnASingleLine =
392-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
393-
/*Inline=*/true,
394-
/*Other=*/true});
392+
FormatStyle::ShortFunctionStyle::setAll();
395393
Style.MaxEmptyLinesToKeep = 2;
396394
Style.BreakBeforeBraces = FormatStyle::BS_Custom;
397395
Style.BraceWrapping.AfterClass = true;
@@ -3381,9 +3379,7 @@ TEST_F(FormatTest, MultiLineControlStatements) {
33813379
Style.BraceWrapping.AfterStruct = false;
33823380
Style.BraceWrapping.AfterControlStatement = FormatStyle::BWACS_MultiLine;
33833381
Style.AllowShortFunctionsOnASingleLine =
3384-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
3385-
/*Inline=*/true,
3386-
/*Other=*/true});
3382+
FormatStyle::ShortFunctionStyle::setAll();
33873383
Style.ColumnLimit = 80;
33883384
verifyFormat("void shortfunction() { bar(); }", Style);
33893385
verifyFormat("struct T shortfunction() { return bar(); }", Style);
@@ -3405,9 +3401,7 @@ TEST_F(FormatTest, MultiLineControlStatements) {
34053401
Style.BraceWrapping.AfterFunction = false;
34063402
Style.BraceWrapping.AfterStruct = true;
34073403
Style.AllowShortFunctionsOnASingleLine =
3408-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
3409-
/*Inline=*/true,
3410-
/*Other=*/true});
3404+
FormatStyle::ShortFunctionStyle::setAll();
34113405
verifyFormat("void shortfunction() { bar(); }", Style);
34123406
verifyFormat("struct T shortfunction() { return bar(); }", Style);
34133407
verifyFormat("struct T\n"
@@ -4210,9 +4204,7 @@ TEST_F(FormatTest, FormatsNamespaces) {
42104204
FormatStyle ShortInlineFunctions = getLLVMStyle();
42114205
ShortInlineFunctions.NamespaceIndentation = FormatStyle::NI_All;
42124206
ShortInlineFunctions.AllowShortFunctionsOnASingleLine =
4213-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
4214-
/*Inline=*/true,
4215-
/*Other=*/false});
4207+
FormatStyle::ShortFunctionStyle::setEmptyAndInline();
42164208
verifyFormat("namespace {\n"
42174209
" void f() {\n"
42184210
" return;\n"
@@ -8349,9 +8341,7 @@ TEST_F(FormatTest, BreakConstructorInitializersAfterColon) {
83498341

83508342
Style.ColumnLimit = 80;
83518343
Style.AllowShortFunctionsOnASingleLine =
8352-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
8353-
/*Inline=*/true,
8354-
/*Other=*/true});
8344+
FormatStyle::ShortFunctionStyle::setAll();
83558345
Style.ConstructorInitializerIndentWidth = 2;
83568346
verifyFormat("SomeClass::Constructor() : a(a), b(b), c(c) {}", Style);
83578347
verifyFormat("SomeClass::Constructor() :\n"
@@ -15007,9 +14997,7 @@ TEST_F(FormatTest, PullTrivialFunctionDefinitionsIntoSingleLine) {
1500714997
TEST_F(FormatTest, PullEmptyFunctionDefinitionsIntoSingleLine) {
1500814998
FormatStyle MergeEmptyOnly = getLLVMStyle();
1500914999
MergeEmptyOnly.AllowShortFunctionsOnASingleLine =
15010-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
15011-
/*Inline=*/false,
15012-
/*Other=*/false});
15000+
FormatStyle::ShortFunctionStyle::setEmptyOnly();
1501315001
verifyFormat("class C {\n"
1501415002
" int f() {}\n"
1501515003
"};",
@@ -15039,9 +15027,7 @@ TEST_F(FormatTest, PullEmptyFunctionDefinitionsIntoSingleLine) {
1503915027
TEST_F(FormatTest, PullInlineFunctionDefinitionsIntoSingleLine) {
1504015028
FormatStyle MergeInlineOnly = getLLVMStyle();
1504115029
MergeInlineOnly.AllowShortFunctionsOnASingleLine =
15042-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
15043-
/*Inline=*/true,
15044-
/*Other=*/false});
15030+
FormatStyle::ShortFunctionStyle::setEmptyAndInline();
1504515031
verifyFormat("class C {\n"
1504615032
" int f() { return 42; }\n"
1504715033
"};",
@@ -15146,9 +15132,7 @@ TEST_F(FormatTest, PullInlineFunctionDefinitionsIntoSingleLine) {
1514615132
TEST_F(FormatTest, CustomShortFunctionOptions) {
1514715133
FormatStyle CustomEmpty = getLLVMStyle();
1514815134
CustomEmpty.AllowShortFunctionsOnASingleLine =
15149-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
15150-
/*Inline=*/false,
15151-
/*Other=*/false});
15135+
FormatStyle::ShortFunctionStyle::setEmptyOnly();
1515215136

1515315137
// Empty functions should be on a single line
1515415138
verifyFormat("int f() {}", CustomEmpty);
@@ -15185,9 +15169,7 @@ TEST_F(FormatTest, CustomShortFunctionOptions) {
1518515169
// Test with Inline = true, All = false
1518615170
FormatStyle CustomInline = getLLVMStyle();
1518715171
CustomInline.AllowShortFunctionsOnASingleLine =
15188-
FormatStyle::ShortFunctionStyle({/*Empty=*/false,
15189-
/*Inline=*/true,
15190-
/*Other=*/false});
15172+
FormatStyle::ShortFunctionStyle::setInlineOnly();
1519115173

1519215174
verifyFormat("class C {\n"
1519315175
" int f() {}\n"
@@ -15212,9 +15194,7 @@ TEST_F(FormatTest, CustomShortFunctionOptions) {
1521215194
// Test with All = true
1521315195
FormatStyle CustomAll = getLLVMStyle();
1521415196
CustomAll.AllowShortFunctionsOnASingleLine =
15215-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
15216-
/*Inline=*/true,
15217-
/*Other=*/true});
15197+
FormatStyle::ShortFunctionStyle::setAll();
1521815198

1521915199
// All functions should be on a single line if they fit
1522015200
verifyFormat("int f() { return 42; }", CustomAll);
@@ -15233,9 +15213,7 @@ TEST_F(FormatTest, CustomShortFunctionOptions) {
1523315213
// Test various combinations
1523415214
FormatStyle CustomMixed = getLLVMStyle();
1523515215
CustomMixed.AllowShortFunctionsOnASingleLine =
15236-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
15237-
/*Inline=*/true,
15238-
/*Other=*/false});
15216+
FormatStyle::ShortFunctionStyle::setEmptyAndInline();
1523915217

1524015218
// Empty functions should be on a single line
1524115219
verifyFormat("int f() {}", CustomMixed);
@@ -15260,9 +15238,7 @@ TEST_F(FormatTest, CustomShortFunctionOptions) {
1526015238
TEST_F(FormatTest, PullInlineOnlyFunctionDefinitionsIntoSingleLine) {
1526115239
FormatStyle MergeInlineOnly = getLLVMStyle();
1526215240
MergeInlineOnly.AllowShortFunctionsOnASingleLine =
15263-
FormatStyle::ShortFunctionStyle({/*Empty=*/false,
15264-
/*Inline=*/true,
15265-
/*Other=*/false});
15241+
FormatStyle::ShortFunctionStyle::setInlineOnly();
1526615242
verifyFormat("class C {\n"
1526715243
" int f() { return 42; }\n"
1526815244
"};",
@@ -15334,9 +15310,7 @@ TEST_F(FormatTest, SplitEmptyFunction) {
1533415310
Style);
1533515311

1533615312
Style.AllowShortFunctionsOnASingleLine =
15337-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
15338-
/*Inline=*/false,
15339-
/*Other=*/false});
15313+
FormatStyle::ShortFunctionStyle::setEmptyOnly();
1534015314
verifyFormat("int f() {}", Style);
1534115315
verifyFormat("int aaaaaaaaaaaaaa(int bbbbbbbbbbbbbb)\n"
1534215316
"{}",
@@ -15348,9 +15322,7 @@ TEST_F(FormatTest, SplitEmptyFunction) {
1534815322
Style);
1534915323

1535015324
Style.AllowShortFunctionsOnASingleLine =
15351-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
15352-
/*Inline=*/true,
15353-
/*Other=*/false});
15325+
FormatStyle::ShortFunctionStyle::setEmptyAndInline();
1535415326
verifyFormat("class Foo {\n"
1535515327
" int f() {}\n"
1535615328
"};",
@@ -15359,6 +15331,10 @@ TEST_F(FormatTest, SplitEmptyFunction) {
1535915331
" int f() { return 0; }\n"
1536015332
"};",
1536115333
Style);
15334+
verifyFormat("class Foo {\n"
15335+
" int f() { return 0; }\n"
15336+
"};",
15337+
Style);
1536215338
verifyFormat("class Foo {\n"
1536315339
" int aaaaaaaaaaaaaa(int bbbbbbbbbbbbbb)\n"
1536415340
" {}\n"
@@ -15373,9 +15349,7 @@ TEST_F(FormatTest, SplitEmptyFunction) {
1537315349
Style);
1537415350

1537515351
Style.AllowShortFunctionsOnASingleLine =
15376-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
15377-
/*Inline=*/true,
15378-
/*Other=*/true});
15352+
FormatStyle::ShortFunctionStyle::setAll();
1537915353
verifyFormat("int f() {}", Style);
1538015354
verifyFormat("int f() { return 0; }", Style);
1538115355
verifyFormat("int aaaaaaaaaaaaaa(int bbbbbbbbbbbbbb)\n"
@@ -15421,9 +15395,7 @@ TEST_F(FormatTest, SplitEmptyFunctionButNotRecord) {
1542115395
TEST_F(FormatTest, KeepShortFunctionAfterPPElse) {
1542215396
FormatStyle Style = getLLVMStyle();
1542315397
Style.AllowShortFunctionsOnASingleLine =
15424-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
15425-
/*Inline=*/true,
15426-
/*Other=*/true});
15398+
FormatStyle::ShortFunctionStyle::setAll();
1542715399
verifyFormat("#ifdef A\n"
1542815400
"int f() {}\n"
1542915401
"#else\n"
@@ -21546,9 +21518,7 @@ TEST_F(FormatTest, WhitesmithsBraceBreaking) {
2154621518

2154721519
// Make a few changes to the style for testing purposes
2154821520
WhitesmithsBraceStyle.AllowShortFunctionsOnASingleLine =
21549-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
21550-
/*Inline=*/false,
21551-
/*Other=*/false});
21521+
FormatStyle::ShortFunctionStyle::setEmptyOnly();
2155221522
WhitesmithsBraceStyle.AllowShortLambdasOnASingleLine = FormatStyle::SLS_None;
2155321523

2155421524
// FIXME: this test case can't decide whether there should be a blank line
@@ -23105,9 +23075,7 @@ TEST_F(FormatTest, BreakConstructorInitializersBeforeComma) {
2310523075

2310623076
Style.ColumnLimit = 80;
2310723077
Style.AllowShortFunctionsOnASingleLine =
23108-
FormatStyle::ShortFunctionStyle({/*Empty=*/true,
23109-
/*Inline=*/true,
23110-
/*Other=*/true});
23078+
FormatStyle::ShortFunctionStyle::setAll();
2311123079
Style.ConstructorInitializerIndentWidth = 2;
2311223080
verifyFormat("SomeClass::Constructor()\n"
2311323081
" : a(a)\n"

0 commit comments

Comments
 (0)