[PATCH] D50571: [clangd] add an extension field to LSP to transfer the diagnostic's category

Tue Aug 21 17:59:46 PDT 2018

joaotavora added a comment.

In https://reviews.llvm.org/D50571#1206020, @arphaman wrote:

> In https://reviews.llvm.org/D50571#1205650, @joaotavora wrote:
>
> > > LGTM. Let's watch out for possible breakages in any of the clients, though. I've checked VSCode and it seems to be fine with the added fields.
> >
> > This isn't in the spec and broke the LSP client eglot (https://github.com/joaotavora/eglot/pull/81). Why don't you put this in the "source" field, or concat it to the "message" field.  Who can even use this information if it's not in the spec? Are clients supposed to code against every LSP server's whim?
>
>
> Thanks for the feedback. I'll make a patch that turns this off by default so that clients can opt-in into it.

Thank you very much, and sorry if I came across a bit hostile.  What is this category field good for?

Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D50571