Skip to content

Commit a3b4d91

Browse files
committed
Fix -Wdangling-assignment-gsl in ClangdLSPServerTests
I think this is a false positive for a non-capturing lambda, but I can't find anything in the standard that guarantees that these have eternal lifetime.
1 parent fe162be commit a3b4d91

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

clang-tools-extra/clangd/unittests/ClangdLSPServerTests.cpp

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -208,12 +208,13 @@ TEST_F(LSPTest, ClangTidyRename) {
208208
Annotations Source(R"cpp(
209209
void [[foo]]() {}
210210
)cpp");
211-
Opts.ClangTidyProvider = [](tidy::ClangTidyOptions &ClangTidyOpts,
212-
llvm::StringRef) {
211+
constexpr auto ClangTidyProvider = [](tidy::ClangTidyOptions &ClangTidyOpts,
212+
llvm::StringRef) {
213213
ClangTidyOpts.Checks = {"-*,readability-identifier-naming"};
214214
ClangTidyOpts.CheckOptions["readability-identifier-naming.FunctionCase"] =
215215
"CamelCase";
216216
};
217+
Opts.ClangTidyProvider = ClangTidyProvider;
217218
auto &Client = start();
218219
Client.didOpen("foo.hpp", Header.code());
219220
Client.didOpen("foo.cpp", Source.code());
@@ -266,10 +267,11 @@ TEST_F(LSPTest, ClangTidyCrash_Issue109367) {
266267
// This test requires clang-tidy checks to be linked in.
267268
if (!CLANGD_TIDY_CHECKS)
268269
return;
269-
Opts.ClangTidyProvider = [](tidy::ClangTidyOptions &ClangTidyOpts,
270-
llvm::StringRef) {
270+
constexpr auto ClangTidyProvider = [](tidy::ClangTidyOptions &ClangTidyOpts,
271+
llvm::StringRef) {
271272
ClangTidyOpts.Checks = {"-*,boost-use-ranges"};
272273
};
274+
Opts.ClangTidyProvider = ClangTidyProvider;
273275
// Check that registering the boost-use-ranges checker's matchers
274276
// on two different threads does not cause a crash.
275277
auto &Client = start();

0 commit comments

Comments
 (0)