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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to