From 60053a9ce28655fc6f635567c62599fa3aad57d2 Mon Sep 17 00:00:00 2001 From: Sam McCall Date: Sun, 31 Jan 2021 12:13:20 +0100 Subject: [PATCH] [clangd] Remove references to old future-based API. NFC --- clang-tools-extra/clangd/ClangdServer.h | 8 +------- clang-tools-extra/clangd/support/Context.h | 2 -- 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/clang-tools-extra/clangd/ClangdServer.h b/clang-tools-extra/clangd/ClangdServer.h index 0a452da..fd5a4bf 100644 --- a/clang-tools-extra/clangd/ClangdServer.h +++ b/clang-tools-extra/clangd/ClangdServer.h @@ -34,7 +34,6 @@ #include "llvm/ADT/Optional.h" #include "llvm/ADT/StringRef.h" #include -#include #include #include #include @@ -194,13 +193,8 @@ public: void removeDocument(PathRef File); /// Run code completion for \p File at \p Pos. - /// Request is processed asynchronously. /// - /// This method should only be called for currently tracked files. However, it - /// is safe to call removeDocument for \p File after this method returns, even - /// while returned future is not yet ready. - /// A version of `codeComplete` that runs \p Callback on the processing thread - /// when codeComplete results become available. + /// This method should only be called for currently tracked files. void codeComplete(PathRef File, Position Pos, const clangd::CodeCompleteOptions &Opts, Callback CB); diff --git a/clang-tools-extra/clangd/support/Context.h b/clang-tools-extra/clangd/support/Context.h index 894032b..815962b 100644 --- a/clang-tools-extra/clangd/support/Context.h +++ b/clang-tools-extra/clangd/support/Context.h @@ -82,8 +82,6 @@ private: public: /// Same as Context::empty(), please use Context::empty() instead. - /// Constructor is defined to workaround a bug in MSVC's version of STL. - /// (arguments of std::future<> must be default-constructible in MSVC). Context() = default; /// Copy operations for this class are deleted, use an explicit clone() method -- 2.7.4