[PATCH] D44408: Move DraftMgr from ClangdServer to ClangdLSPServer

Simon Marchi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 14 11:12:23 PDT 2018


simark added a comment.

In https://reviews.llvm.org/D44408#1037327, @ilya-biryukov wrote:

> In https://reviews.llvm.org/D44408#1036941, @simark wrote:
>
> > I don't see how to avoid adding the Contents parameter to the codeComplete and signatureHelp methods, since they needed to fetch the document from the draft manager and pass it to the backend.
>
>
> You can get the contents inside the `runWithPreamble` callback:
>
>   llvm::Expected<InputsAndPreamble> IP;
>   auto &Contents = IP->Inputs.Contents;
>


I see, I made this as a separate patch:

https://reviews.llvm.org/D44484


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D44408





More information about the cfe-commits mailing list