[PATCH] D58291: [clangd] Include textual diagnostic ID as Diagnostic.code.
Kadir Cetinkaya via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Feb 17 06:50:36 PST 2019
kadircet accepted this revision.
kadircet added a comment.
This revision is now accepted and ready to land.
LG but is this information really useful to users? According to LSP `The diagnostic's code, which might appear in the user interface.`, I think seeing this will be mostly noise for users.
================
Comment at: clangd/Diagnostics.cpp:39
+#include "clang/Basic/DiagnosticCommentKinds.inc"
+#include "clang/Basic/DiagnosticSemaKinds.inc"
+#include "clang/Basic/DiagnosticAnalysisKinds.inc"
----------------
I suppose `CrossTUKinds` is left out intentionally ?
Repository:
rCTE Clang Tools Extra
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58291/new/
https://reviews.llvm.org/D58291
More information about the cfe-commits
mailing list