diff options
| author | Sam McCall <sam.mccall@gmail.com> | 2018-07-02 11:13:16 +0000 |
|---|---|---|
| committer | Sam McCall <sam.mccall@gmail.com> | 2018-07-02 11:13:16 +0000 |
| commit | e746a2b4f7dbe0c6b8dd280a92ddfb1644b232f9 (patch) | |
| tree | 22e2edcfb2a00ed68dce407a698c25dbd7389b69 /clang-tools-extra/unittests/clangd/SyncAPI.h | |
| parent | 2757df7933766a8ec344cb51eebfaa554812c685 (diff) | |
| download | bcm5719-llvm-e746a2b4f7dbe0c6b8dd280a92ddfb1644b232f9.tar.gz bcm5719-llvm-e746a2b4f7dbe0c6b8dd280a92ddfb1644b232f9.zip | |
[clangd] ClangdServer::codeComplete return CodeCompleteResult, not LSP struct.
Summary:
This provides more structured information that embedders can use for rendering.
ClangdLSPServer continues to call render(), so NFC.
The patch is:
- trivial changes to ClangdServer/ClangdLSPServer
- mostly-mechanical updates to CodeCompleteTests etc for the new API
- new direct tests of render() in CodeCompleteTests
- tiny cleanups to CodeCompletionItem (operator<< and missing initializers)
Reviewers: ioeric
Subscribers: ilya-biryukov, MaskRay, jkorous, cfe-commits
Differential Revision: https://reviews.llvm.org/D48821
llvm-svn: 336094
Diffstat (limited to 'clang-tools-extra/unittests/clangd/SyncAPI.h')
| -rw-r--r-- | clang-tools-extra/unittests/clangd/SyncAPI.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang-tools-extra/unittests/clangd/SyncAPI.h b/clang-tools-extra/unittests/clangd/SyncAPI.h index 0a140aa9db3..8b325d31394 100644 --- a/clang-tools-extra/unittests/clangd/SyncAPI.h +++ b/clang-tools-extra/unittests/clangd/SyncAPI.h @@ -22,7 +22,7 @@ namespace clangd { void runAddDocument(ClangdServer &Server, PathRef File, StringRef Contents, WantDiagnostics WantDiags = WantDiagnostics::Auto); -llvm::Expected<CompletionList> +llvm::Expected<CodeCompleteResult> runCodeComplete(ClangdServer &Server, PathRef File, Position Pos, clangd::CodeCompleteOptions Opts); |
