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



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

Reply via email to