[PATCH] D44462: [clangd] Don't use DraftMgr in implementation of forceReparse.
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 14 09:59:12 PDT 2018
sammccall accepted this revision.
sammccall added inline comments.
Comment at: clangd/TUScheduler.cpp:220
+ FileInputs = std::move(Inputs);
+ rebuildASTLocked(WantDiags, std::move(OnUpdated));
you might want to inline this function here again - it's not overly long, and we're now back to one callsite
Comment at: unittests/clangd/SyncAPI.h:23
+void runAddDocument(ClangdServer &Server, PathRef File, StringRef Contents,
+ bool SkipCache = false);
it's slightly odd that wantdiagnostics is missing here. Previously this was "forwards compatible" and could easily be added later. But now we'll have callsites that pass different arguments than addDocument takes.
Maybe just add the defaulted param to the signature?
rCTE Clang Tools Extra
More information about the cfe-commits