[PATCH] D58291: [clangd] Include textual diagnostic ID as Diagnostic.code.
Kadir Cetinkaya via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 17 02:55:47 PDT 2019
kadircet accepted this revision.
kadircet added a comment.
In D58291#1469880 <https://reviews.llvm.org/D58291#1469880>, @sammccall wrote:
> It's a good question, it depends how this is surfaced, and we may want to tweak the behavior or suppress entirely in some cases.
> I think at least some are useful:
> - clang-tidy check names are things users need to know about (used for configuration)
> - for warnings, we quite likely should replace with the most specific warning category (e.g. "unreachable-code-loop-increment"), again these are used for configuration (-W)
> - for others, maybe we should at least trim the err_ prefix, or maybe drop them entirely.
I see, I believe we can decide on what tweaks to perform after landing this patch and seeing behaviors in different editors, but up to you.
As for the lit-tests, it would be great to have a diag with source clang-tidy if it is not too much of a hustle.
rCTE Clang Tools Extra
CHANGES SINCE LAST ACTION
More information about the cfe-commits