Skip to content

Commit f3a3270

Browse files
committed
[clang-format][NFC] Delete redundant type names after FormatStyle::
1 parent 853c343 commit f3a3270

File tree

2 files changed

+12
-14
lines changed

2 files changed

+12
-14
lines changed

clang/unittests/Format/FormatTest.cpp

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -3185,7 +3185,7 @@ TEST_F(FormatTest, FormatsLabels) {
31853185
// The opening brace may either be on the same unwrapped line as the colon or
31863186
// on a separate one. The formatter should recognize both.
31873187
Style = getLLVMStyle();
3188-
Style.BreakBeforeBraces = FormatStyle::BraceBreakingStyle::BS_Allman;
3188+
Style.BreakBeforeBraces = FormatStyle::BS_Allman;
31893189
verifyFormat("{\n"
31903190
" some_code();\n"
31913191
"test_label:\n"
@@ -3206,7 +3206,7 @@ TEST_F(FormatTest, FormatsLabels) {
32063206

32073207
TEST_F(FormatTest, MultiLineControlStatements) {
32083208
FormatStyle Style = getLLVMStyleWithColumns(20);
3209-
Style.BreakBeforeBraces = FormatStyle::BraceBreakingStyle::BS_Custom;
3209+
Style.BreakBeforeBraces = FormatStyle::BS_Custom;
32103210
Style.BraceWrapping.AfterControlStatement = FormatStyle::BWACS_MultiLine;
32113211
// Short lines should keep opening brace on same line.
32123212
verifyFormat("if (foo) {\n"
@@ -3441,7 +3441,7 @@ TEST_F(FormatTest, MultiLineControlStatements) {
34413441

34423442
TEST_F(FormatTest, BeforeWhile) {
34433443
FormatStyle Style = getLLVMStyle();
3444-
Style.BreakBeforeBraces = FormatStyle::BraceBreakingStyle::BS_Custom;
3444+
Style.BreakBeforeBraces = FormatStyle::BS_Custom;
34453445

34463446
verifyFormat("do {\n"
34473447
" foo();\n"
@@ -23779,7 +23779,7 @@ TEST_F(FormatTest, FormatsLambdas) {
2377923779
LLVMWithBeforeLambdaBody.BreakBeforeBraces = FormatStyle::BS_Custom;
2378023780
LLVMWithBeforeLambdaBody.BraceWrapping.BeforeLambdaBody = true;
2378123781
LLVMWithBeforeLambdaBody.AllowShortLambdasOnASingleLine =
23782-
FormatStyle::ShortLambdaStyle::SLS_None;
23782+
FormatStyle::SLS_None;
2378323783
verifyFormat("FctWithOneNestedLambdaInline_SLS_None(\n"
2378423784
" []()\n"
2378523785
" {\n"
@@ -23815,7 +23815,7 @@ TEST_F(FormatTest, FormatsLambdas) {
2381523815
LLVMWithBeforeLambdaBody);
2381623816

2381723817
LLVMWithBeforeLambdaBody.AllowShortLambdasOnASingleLine =
23818-
FormatStyle::ShortLambdaStyle::SLS_Empty;
23818+
FormatStyle::SLS_Empty;
2381923819
verifyFormat("FctWithOneNestedLambdaInline_SLS_Empty(\n"
2382023820
" []()\n"
2382123821
" {\n"
@@ -23862,7 +23862,7 @@ TEST_F(FormatTest, FormatsLambdas) {
2386223862
LLVMWithBeforeLambdaBody);
2386323863

2386423864
LLVMWithBeforeLambdaBody.AllowShortLambdasOnASingleLine =
23865-
FormatStyle::ShortLambdaStyle::SLS_Inline;
23865+
FormatStyle::SLS_Inline;
2386623866
verifyFormat("FctWithOneNestedLambdaInline_SLS_Inline([]() { return 17; });",
2386723867
LLVMWithBeforeLambdaBody);
2386823868
verifyFormat("FctWithOneNestedLambdaEmpty_SLS_Inline([]() {});",
@@ -23893,7 +23893,7 @@ TEST_F(FormatTest, FormatsLambdas) {
2389323893
LLVMWithBeforeLambdaBody);
2389423894

2389523895
LLVMWithBeforeLambdaBody.AllowShortLambdasOnASingleLine =
23896-
FormatStyle::ShortLambdaStyle::SLS_All;
23896+
FormatStyle::SLS_All;
2389723897
verifyFormat("FctWithOneNestedLambdaInline_SLS_All([]() { return 17; });",
2389823898
LLVMWithBeforeLambdaBody);
2389923899
verifyFormat("FctWithOneNestedLambdaEmpty_SLS_All([]() {});",
@@ -24025,7 +24025,7 @@ TEST_F(FormatTest, FormatsLambdas) {
2402524025
LLVMWithBeforeLambdaBody);
2402624026

2402724027
LLVMWithBeforeLambdaBody.AllowShortLambdasOnASingleLine =
24028-
FormatStyle::ShortLambdaStyle::SLS_None;
24028+
FormatStyle::SLS_None;
2402924029

2403024030
verifyFormat("auto select = [this]() -> const Library::Object *\n"
2403124031
"{\n"
@@ -24273,7 +24273,7 @@ TEST_F(FormatTest, LambdaWithLineComments) {
2427324273
LLVMWithBeforeLambdaBody.BreakBeforeBraces = FormatStyle::BS_Custom;
2427424274
LLVMWithBeforeLambdaBody.BraceWrapping.BeforeLambdaBody = true;
2427524275
LLVMWithBeforeLambdaBody.AllowShortLambdasOnASingleLine =
24276-
FormatStyle::ShortLambdaStyle::SLS_All;
24276+
FormatStyle::SLS_All;
2427724277

2427824278
verifyFormat("auto k = []() { return; }", LLVMWithBeforeLambdaBody);
2427924279
verifyFormat("auto k = []() // comment\n"
@@ -28371,7 +28371,7 @@ TEST_F(FormatTest, BreakAfterAttributes) {
2837128371
"Foo &operator-(Foo &);",
2837228372
Style);
2837328373

28374-
Style.ReferenceAlignment = FormatStyle::ReferenceAlignmentStyle::RAS_Left;
28374+
Style.ReferenceAlignment = FormatStyle::RAS_Left;
2837528375
verifyFormat("[[nodiscard]]\n"
2837628376
"Foo& operator-(Foo&);",
2837728377
Style);

clang/unittests/Format/FormatTestSelective.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -689,8 +689,7 @@ TEST_F(FormatTestSelective, FormatMacroRegardlessOfPreviousIndent) {
689689
" }};", // Ditto: Bug?
690690
format(Code, 57, 0));
691691

692-
Style.IndentPPDirectives =
693-
FormatStyle::PPDirectiveIndentStyle::PPDIS_BeforeHash;
692+
Style.IndentPPDirectives = FormatStyle::PPDIS_BeforeHash;
694693
EXPECT_EQ(" class Foo {\n"
695694
" void test() {\n"
696695
" #ifdef 1\n"
@@ -699,8 +698,7 @@ TEST_F(FormatTestSelective, FormatMacroRegardlessOfPreviousIndent) {
699698
" }};",
700699
format(Code, 57, 0));
701700

702-
Style.IndentPPDirectives =
703-
FormatStyle::PPDirectiveIndentStyle::PPDIS_AfterHash;
701+
Style.IndentPPDirectives = FormatStyle::PPDIS_AfterHash;
704702
EXPECT_EQ(" class Foo {\n"
705703
" void test() {\n"
706704
" #ifdef 1\n"

0 commit comments

Comments
 (0)