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



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

Reply via email to