[PATCH] D44226: [clangd] Add -log-lsp-to-stderr option

[Marc-Andre Laperle via Phabricator via cfe-commits]

malaperle added a comment.
Herald added a subscriber: MaskRay.

Gentle ping! It would be nice to have this so make Clangd less "verbose".



================
Comment at: clangd/ClangdLSPServer.cpp:412
+    llvm::raw_string_ostream OS(Message);
+    OS << "method not found (" << Method << ")";
+    replyError(ErrorCode::MethodNotFound, OS.str());
----------------
ilya-biryukov wrote:
> Could we also `log` (i.e. not `vlog`) names of the methods that were handled successfully? To have some context when something crashes and we only have non-verbose logs.
Wouldn't that mean pretty much logging everything coming in? The idea of this patch it to make it that by default Clangd is not talkative unless there is an error and optionally make it verbose, for troubleshooting.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D44226