Add support for graying out unused imports

1 files changed, 35 insertions, 30 deletions

diff --git a/src/server/documents.odin b/src/server/documents.odin
index cf8a344..591c56c 100644
--- a/src/server/documents.odin
+++ b/src/server/documents.odin
@@ -76,7 +76,7 @@ document_get_allocator :: proc() -> ^virtual.Arena {
 		return pop(&document_storage.free_allocators)
 	} else {
 		allocator := new(virtual.Arena)
-		_ = virtual.arena_init_growing(allocator) 
+		_ = virtual.arena_init_growing(allocator)
 		return allocator
 	}
 }
@@ -319,26 +319,50 @@ document_refresh :: proc(document: ^Document, config: ^common.Config, writer: ^W
 		return .None
 	}
 
-	if writer != nil && len(errors) > 0 && !config.disable_parser_errors {
+	if writer != nil && !config.disable_parser_errors {
 		document.diagnosed_errors = true
 
+		diagnostics := make([dynamic]Diagnostic, 0, len(errors), context.temp_allocator)
+
 		params := NotificationPublishDiagnosticsParams {
-			uri         = document.uri.uri,
-			diagnostics = make([]Diagnostic, len(errors), context.temp_allocator),
+			uri = document.uri.uri,
 		}
 
 		for error, i in errors {
-			params.diagnostics[i] = Diagnostic {
-				range = common.Range {
-					start = common.Position{line = error.line - 1, character = 0},
-					end = common.Position{line = error.line, character = 0},
+			append(
+				&diagnostics,
+				Diagnostic {
+					range = common.Range {
+						start = common.Position{line = error.line - 1, character = 0},
+						end = common.Position{line = error.line, character = 0},
+					},
+					severity = DiagnosticSeverity.Error,
+					code = "Syntax",
+					message = error.message,
 				},
-				severity = DiagnosticSeverity.Error,
-				code = "Syntax",
-				message = error.message,
+			)
+		}
+
+		if config.enable_unused_imports_reporting {
+			unused_imports := find_unused_imports(document, context.temp_allocator)
+
+			for imp in unused_imports {
+				append(
+					&diagnostics,
+					Diagnostic {
+						range = common.get_token_range(imp.import_decl, document.ast.src),
+						severity = DiagnosticSeverity.Hint,
+						code = "Unused",
+						message = "unused import",
+						tags = {.Unnecessary},
+					},
+				)
 			}
+
 		}
 
+		params.diagnostics = diagnostics[:]
+
 		notifaction := Notification {
 			jsonrpc = "2.0",
 			method  = "textDocument/publishDiagnostics",
@@ -348,25 +372,6 @@ document_refresh :: proc(document: ^Document, config: ^common.Config, writer: ^W
 		send_notification(notifaction, writer)
 	}
 
-	if writer != nil && len(errors) == 0 {
-		//send empty diagnosis to remove the clients errors
-		if document.diagnosed_errors {
-
-			notifaction := Notification {
-				jsonrpc = "2.0",
-				method = "textDocument/publishDiagnostics",
-				params = NotificationPublishDiagnosticsParams {
-					uri = document.uri.uri,
-					diagnostics = make([]Diagnostic, len(errors), context.temp_allocator),
-				},
-			}
-
-			document.diagnosed_errors = false
-
-			send_notification(notifaction, writer)
-		}
-	}
-
 	return .None
 }