[PATCH] D65373: [clangd] Update features table in the docs with links to LSP extension proposals

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 12 01:25:07 PDT 2019


ilya-biryukov added inline comments.


================
Comment at: clang-tools-extra/docs/clangd/Features.rst:260
 +-------------------------------------+------------+----------+
-| Syntax and Semantic Coloring        | No         |   No     |
+| Syntax and Semantic Coloring        |`Proposed`__|   No     |
 +-------------------------------------+------------+----------+
----------------
Could you mention that semantic coloring is implemented in clangd? (with a footnote that it only works in `Theia` atm)


================
Comment at: clang-tools-extra/docs/clangd/Features.rst:269
+
+__ https://github.com/microsoft/language-server-protocol/issues/18
+__ https://github.com/microsoft/language-server-protocol/issues/468
----------------
Having the same marker for three links makes is hard to read. WDYT about using the footnote syntax here?
```
Proposed [1]_
Proposed [2]_


.. [1] https://github.com/microsoft/language-server-protocol/issues/468
.. [2] https://github.com/microsoft/language-server-protocol/issues/136
```

Just a suggestion, not sure which of the two versions renders nicer.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65373/new/

https://reviews.llvm.org/D65373





More information about the cfe-commits mailing list