[PATCH] D59757: [clangd] Send empty diagnostics when a file is closed

[Haojian Wu via Phabricator via cfe-commits]

hokein added inline comments.


================
Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:1000
     std::lock_guard<std::mutex> Lock(FixItsMutex);
     FixItsMap[File] = LocalFixIts;
   }
----------------
IIUC, it seems that we might have race condition here, considering:

- open a file which will take a long time to get diagnostic
- the file gets closed, we publish an empty diagnostic
- we get the diagnostics of the file, and we cache them, and emit them to clients


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59757