Browse Source

Include gdscript warning name in LSP message.

My initial attempt changed this in the gdscript code, which resulted in
a duplicate warning name in the builtin editor. We should just append
the warning name in the LSP instead.

This uses parens to match what is shown in the builtin editor.

(cherry picked from commit 8dcc39ec91babb282bff69c80bb7f2bde657677f)
Ryan Roden-Corrent 5 years ago
parent
commit
4677502d7c
1 changed files with 1 additions and 1 deletions
  1. 1 1
      modules/gdscript/language_server/gdscript_extend_parser.cpp

+ 1 - 1
modules/gdscript/language_server/gdscript_extend_parser.cpp

@@ -63,7 +63,7 @@ void ExtendGDScriptParser::update_diagnostics() {
 		const GDScriptWarning &warning = E->get();
 		const GDScriptWarning &warning = E->get();
 		lsp::Diagnostic diagnostic;
 		lsp::Diagnostic diagnostic;
 		diagnostic.severity = lsp::DiagnosticSeverity::Warning;
 		diagnostic.severity = lsp::DiagnosticSeverity::Warning;
-		diagnostic.message = warning.get_message();
+		diagnostic.message = "(" + warning.get_name() + "): " + warning.get_message();
 		diagnostic.source = "gdscript";
 		diagnostic.source = "gdscript";
 		diagnostic.code = warning.code;
 		diagnostic.code = warning.code;
 		lsp::Range range;
 		lsp::Range range;