[PATCH] D52937: [clangd] Add clangd.serverInfo command to inspect server state.

2018-10-08 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. Rather than use a `workspace/executeCommand` command, you may consider making it a custom request message whose method name starts with `$/` https://microsoft.github.io/language-server-protocol/specification > Notification and requests whose methods start with ‘$/’ are

[PATCH] D52937: [clangd] Add clangd.serverInfo command to inspect server state.

2018-10-08 Thread Eric Liu via Phabricator via cfe-commits
ioeric added a comment. How will this command be triggered? What's the advantage comparing to `vlog`ing? Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D52937 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.

[PATCH] D52937: [clangd] Add clangd.serverInfo command to inspect server state.

2018-10-05 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: ioeric. Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Initially just export the information that's easily available. (I want to measure changes in dynamic index size, so this is good enough