[PATCH] D81456: [clangd] Get rid of WantDiagnostics::Yes

[Sam McCall via Phabricator via cfe-commits]

sammccall added a comment.

Sorry for the long delay here...



================
Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:670
   auto Version = DraftMgr.addDraft(File, Params.textDocument.version, Contents);
-  Server->addDocument(File, Contents, encodeVersion(Version),
-                      WantDiagnostics::Yes);
+  Server->addDocument(File, Contents, encodeVersion(Version));
 }
----------------
This cuts against the assertion in the description that there are no uses!

And I believe the debounce test demonstrates that that we will in fact debounce an initial request that's `Auto`, right?
So isn't this going to add 0.5sec to time-before-first-diagnostics in all cases?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81456