[PATCH] D54077: [clangd] Implemented DraftFileSystem

[Eric Liu via Phabricator via cfe-commits]

ioeric added a comment.

In https://reviews.llvm.org/D54077#1288387, @sammccall wrote:

> Someone mentioned to me that the interaction-between-features argument wasn't clear here:
>
> - we **don't** currently update diagnostics for `A.cc` when `A.h` is edited
> - we should, this seems more obvious & important than what we do with drafts
> - this interacts badly with using draft state, as this patch proposes - there are too many edits


FWIW, one of my "pain points" when using vim+clangd is:

- Edit `A.h` in a buffer (and forget to save)
- Switch to `A.cc` in another buffer
- Realize that I forgot to save `A.h`
- Go back to save `A.h`
- Jump back to `A.cc` file.

I would be very happy if `A.cc` can see the unsaved `A.h` when I am editing `A.cc`. I think the update should be relatively less frequent with this approach, as people don't usually update multiple files at the same time.  Not sure if I want all files that depend on `A.h` to be updated when I edit `A.h` though; it seems much more complicated and less important (to me).


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D54077