[PATCH] D35215: [clangd] Fix Go to Definition not working in VSCode extension

[Krasimir Georgiev via cfe-commits]

Not really. It will get confused by a setup in which you open 2 files
depending on each other with spaces in their names, for example: A A:
#include "B B", because clangd will not recognize that the local B%20B is
the same as B B.

On Tue, Jul 11, 2017 at 7:35 PM, Marc-Andre Laperle via Phabricator <
reviews at reviews.llvm.org> wrote:

> malaperle added a comment.
>
> In the other review: "The tool is fine with spaces encoded as %20 by the
> way." If that's true, wouldn't that mean the other review is the way to go?
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D35215
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170711/0c46445c/attachment.html>