This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Clean the cache of file statuses on vscode-clangd when clangd crashes.
ClosedPublic

Authored by hokein on Jan 10 2019, 5:14 AM.

Diff Detail

Event Timeline

hokein created this revision.Jan 10 2019, 5:14 AM
ilya-biryukov accepted this revision.Jan 22 2019, 7:28 AM

LGTM

clangd/clients/clangd-vscode/src/extension.ts
127

This looks scary, if we end up doing that in too many places, the code would become completely unreadable.

Have no idea how to make it better, but a very general observation is that we'd be better off with dropping all the state we have when clangd crashes and starting from scratch, rather than trying to keep old components and bring them into a sensible state when crashes happen.

No need to do anything right now, just something to keep in mind for the future.

This revision is now accepted and ready to land.Jan 22 2019, 7:28 AM
This revision was automatically updated to reflect the committed changes.
hokein marked an inline comment as done.Jan 24 2019, 6:27 AM
hokein added inline comments.
clangd/clients/clangd-vscode/src/extension.ts
127

Acked.