[PATCH] D39057: [clangd][WIP] Integrate the refactoring actions into clangd

Alex Lorenz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 27 14:02:02 PDT 2017


arphaman added a comment.

In https://reviews.llvm.org/D39057#907820, @sammccall wrote:

> In https://reviews.llvm.org/D39057#906297, @ilya-biryukov wrote:
>
> > There's another patch (https://reviews.llvm.org/D39276) that tries to add `workspace/executeCommand` for a slightly different use-case.
> >  Could we take the code for parsing/handling `workspace/executeCommand` from this patch and extract it into a separate change so that these two patches can be reviewed independently?
>
>
> @arphaman unless you're far down this path already, I'd actually prefer we land that patch soon and merge this one into it:
>
> - It's got a pretty narrow scope (1 command, not much logic), and will be ready soon. It's almost actually the isolated patch that @ilya-biryukov describes. This patch is both more general in scope and is stacked on top of other nontrivial changes, so will probably take longer.
> - It makes a couple of different decisions that I think would aid the design here, will leave detailed comments.
>
>   But if people prefer, we can certainly tackle command parsing separately.


Ok, I will wait until that patch lands.


Repository:
  rL LLVM

https://reviews.llvm.org/D39057





More information about the cfe-commits mailing list