From 6815037085945be8bb758c23b1a5daabe0a8667d Mon Sep 17 00:00:00 2001 From: Nathan James Date: Fri, 30 Apr 2021 13:27:23 +0100 Subject: [PATCH] [clangd][NFC] Remove unnecessary string captures in lambdas. Due to a somewhat annoying, but necessary, shortfall in -Wunused-lambda-capture, These unused captures aren't warned about. Reviewed By: kadircet Differential Revision: https://reviews.llvm.org/D101611 --- clang-tools-extra/clangd/ClangdLSPServer.cpp | 4 ++-- clang-tools-extra/clangd/ClangdServer.cpp | 6 ++---- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/clang-tools-extra/clangd/ClangdLSPServer.cpp b/clang-tools-extra/clangd/ClangdLSPServer.cpp index 4916dfaafd5a..a7cc1ed42818 100644 --- a/clang-tools-extra/clangd/ClangdLSPServer.cpp +++ b/clang-tools-extra/clangd/ClangdLSPServer.cpp @@ -725,8 +725,8 @@ void ClangdLSPServer::onCommandApplyEdit(const WorkspaceEdit &WE, void ClangdLSPServer::onCommandApplyTweak(const TweakArgs &Args, Callback Reply) { - auto Action = [this, Reply = std::move(Reply), - File = Args.file](llvm::Expected R) mutable { + auto Action = [this, Reply = std::move(Reply)]( + llvm::Expected R) mutable { if (!R) return Reply(R.takeError()); diff --git a/clang-tools-extra/clangd/ClangdServer.cpp b/clang-tools-extra/clangd/ClangdServer.cpp index 11cd1a8fee53..3f8b95b9104c 100644 --- a/clang-tools-extra/clangd/ClangdServer.cpp +++ b/clang-tools-extra/clangd/ClangdServer.cpp @@ -575,8 +575,7 @@ void ClangdServer::enumerateTweaks( // Tracks number of times a tweak has been offered. static constexpr trace::Metric TweakAvailable( "tweak_available", trace::Metric::Counter, "tweak_id"); - auto Action = [File = File.str(), Sel, CB = std::move(CB), - Filter = std::move(Filter), + auto Action = [Sel, CB = std::move(CB), Filter = std::move(Filter), FeatureModules(this->FeatureModules)]( Expected InpAST) mutable { if (!InpAST) @@ -756,8 +755,7 @@ void ClangdServer::incomingCalls( void ClangdServer::inlayHints(PathRef File, Callback> CB) { - auto Action = [File = File.str(), - CB = std::move(CB)](Expected InpAST) mutable { + auto Action = [CB = std::move(CB)](Expected InpAST) mutable { if (!InpAST) return CB(InpAST.takeError()); CB(clangd::inlayHints(InpAST->AST));