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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to