[PATCH] D39571: [clangd] DidChangeConfiguration Notification

Simon Marchi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 23 07:52:38 PST 2018


simark added inline comments.


================
Comment at: clangd/ClangdLSPServer.cpp:78
             {"documentHighlightProvider", true},
+            {"configurationChangeProvider", true},
             {"renameProvider", true},
----------------
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.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D39571





More information about the cfe-commits mailing list