[PATCH] D38629: [clangd] Added a callback-based codeComplete in clangd.
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 13 02:29:09 PDT 2017
sammccall added inline comments.
================
Comment at: clangd/ClangdServer.cpp:51
+template <class Ret, class... Args>
+std::future<Ret> makeFutureAPIFromCallback(
----------------
I'm not sure we need a template here rather than just writing the code directly.
================
Comment at: clangd/ClangdServer.cpp:257
+ // A task that will be run asynchronously.
+ auto Task =
+ // 'mutable' to reassign Preamble variable.
----------------
Isn't this exactly where you'd want to use ForwardBinder?
(Yeah, it looks like the work scheduler itself has this functionality too)
================
Comment at: clangd/ClangdServer.h:261
+ void codeComplete(
+ UniqueFunction<void(Tagged<std::vector<CompletionItem>>)> Callback,
+ PathRef File, Position Pos,
----------------
Hmm, generally having a callback as the last parameter is nicer, but this doesn't play nicely with optional params...
https://reviews.llvm.org/D38629
More information about the cfe-commits
mailing list