malaperle added a comment.

In https://reviews.llvm.org/D35894#830178, @ilya-biryukov wrote:

> In https://reviews.llvm.org/D35894#829190, @malaperle wrote:
>
> > @Nebiroth I think it's OK to put this on hold until we make the "semantic" 
> > hover and figure out how to have both. From our perspective, this is going 
> > beyond parity of what we had before but it's seems like the right thing to 
> > do.
>
>
> BTW, just noticed that VS Code shows the range reported by `findDefinitions` 
> when you hover over a symbol and holding Ctrl.
>  Would calling `findDefinitions` help you to do the same thing? That would 
> allow to get closer to feature parity with CDT without implementing semantic 
> hover in clangd.


That looks very interesting, I had no idea. We'll have to try that. Thanks!


https://reviews.llvm.org/D35894



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

Reply via email to