simark added a comment.

In D55363#1329652 <https://reviews.llvm.org/D55363#1329652>, @ilya-biryukov 
wrote:

> Sorry if I missed any important design discussions here, but wanted to clear 
> up what information we are trying to convey to the user with the status 
> messages?
>  E.g. why seeing "building preamble", "building file" or "queued" in the 
> status bar can be useful to the user? Those messages mention internals of 
> clangd, I'm not sure how someone unfamiliar with internals of clangd should 
> interpret this information.


I agree about not necessarily needing to expose clang internals, but I think 
the end goal of providing some feedback to the user (still thinking/done) is 
worthy.

If this is a clangd-specific event, shouldn't the name of the event reflect it? 
 As-is, it looks like an official LSP feature and can be confusing.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D55363



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

Reply via email to