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

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 3 01:36:35 PST 2019


ilya-biryukov added a comment.

In D55363#1343611 <https://reviews.llvm.org/D55363#1343611>, @jkorous wrote:

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


That would mean more complex code in **every** client and more communication between the server and the client, which is not particularly useful.
Doing it on the server means keeping the complexity in one place, this looks strictly better, unless there are actual use-cases for high-frequency updates.


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