Skip to content

Commit

Permalink
[clangd][test] Fix -Wmissing-field-initializers in DiagnosticsTests.c…
Browse files Browse the repository at this point in the history
…pp (NFC)

llvm-project/clang-tools-extra/clangd/unittests/DiagnosticsTests.cpp:921:45:
error: missing field 'Annotations' initializer [-Werror,-Wmissing-field-initializers]
       TextEdit{Main.range("virtual1"), ""}}};
                                            ^
llvm-project/clang-tools-extra/clangd/unittests/DiagnosticsTests.cpp:926:45:
error: missing field 'Annotations' initializer [-Werror,-Wmissing-field-initializers]
       TextEdit{Main.range("virtual2"), ""}}};
                                            ^
2 errors generated.
  • Loading branch information
DamonFool committed Feb 15, 2024
1 parent 9a1e637 commit 8ce1448
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions clang-tools-extra/clangd/unittests/DiagnosticsTests.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -918,12 +918,14 @@ TEST(DiagnosticTest, ClangTidySelfContainedDiagsFormatting) {
"prefer using 'override' or (rarely) 'final' "
"instead of 'virtual'",
{TextEdit{Main.range("override1"), " override"},
TextEdit{Main.range("virtual1"), ""}}};
TextEdit{Main.range("virtual1"), ""}},
{}};
clangd::Fix const ExpectedFix2{
"prefer using 'override' or (rarely) 'final' "
"instead of 'virtual'",
{TextEdit{Main.range("override2"), " override"},
TextEdit{Main.range("virtual2"), ""}}};
TextEdit{Main.range("virtual2"), ""}},
{}};
// Note that in the Fix we expect the "virtual" keyword and the following
// whitespace to be deleted
EXPECT_THAT(TU.build().getDiagnostics(),
Expand Down

0 comments on commit 8ce1448

Please sign in to comment.