[PATCH] D55363: [clangd] Expose FileStatus in LSP.

Jan Korous via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 2 07:49:00 PST 2019


jkorous added a comment.

In D55363#1337376 <https://reviews.llvm.org/D55363#1337376>, @hokein wrote:

> In D55363#1336315 <https://reviews.llvm.org/D55363#1336315>, @ilya-biryukov wrote:
>
> > Maybe consider sending an update after some period of time, e.g. `0.5s`? (I expect this particular strategy to be a bit complicated and out of scope of this patch, so another alternative is to simply not send them to the clients in the first version). WDYT?
>
>
> Sounds fair, keeping status bar  `blinking` seems annony to users. Added a FIXME.


I am a bit late to the discussion but isn't this something to be dealt with on the client's side?


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D55363





More information about the cfe-commits mailing list