[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