[PATCH] D75582: [clangd] Track document versions, include them with diags, enhance logs

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 4 16:22:48 PST 2020


sammccall marked an inline comment as done.
sammccall added inline comments.


================
Comment at: clang-tools-extra/clangd/ClangdServer.h:73
     virtual void onDiagnosticsReady(PathRef File,
+                                    const llvm::json::Value &Version,
                                     std::vector<Diag> Diagnostics) {}
----------------
kadircet wrote:
> can we rather have `Optional<int>`s here(both for callbacks and `addDocument`)?
> 
> as clangdserver layer doesn't touch json objects at all currently.
I really do want to make these opaque at the lower layer.
json is a bit fiddly though, reworked to use strings instead.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75582/new/

https://reviews.llvm.org/D75582





More information about the cfe-commits mailing list