[PATCH] D50571: [clangd] add an extension field to LSP to transfer the diagnostic's category
Alex Lorenz via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 22 13:31:54 PDT 2018
arphaman added a comment.
In https://reviews.llvm.org/D50571#1208635, @joaotavora wrote:
> 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?
NP! Fixed in r340449.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D50571
More information about the cfe-commits
mailing list