Nebiroth added inline comments.
================
Comment at: clangd/ClangdLSPServer.cpp:78
{"documentHighlightProvider", true},
+ {"configurationChangeProvider", true},
{"renameProvider", true},
----------------
simark wrote:
> I find `configurationChangeProvider` a bit weird. It makes it sound like
> clangd can provide configuration changes. In reality, it can accept
> configuration changes. So I think this should be named something else.
Agreed, perhaps configurationChangeManager would be more appropriate then?
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D39571
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
