Fix spell check endpoint

dotnet · Dec 11, 2023 · cd8cf20 · cd8cf20
1 parent ae0914c
commit cd8cf20
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/...or/src/Microsoft.AspNetCore.Razor.LanguageServer/SpellCheck/DocumentSpellCheckEndpoint.cs b/...or/src/Microsoft.AspNetCore.Razor.LanguageServer/SpellCheck/DocumentSpellCheckEndpoint.cs
@@ -159,10 +159,14 @@ private async Task AddCSharpSpellCheckRangesAsync(List<SpellCheckRange> ranges,
 
                 // We need to map the start index to produce results, and we validate that we can map the end index so we don't have
                 // squiggles that go from C# into Razor/Html.
-                if (_documentMappingService.TryMapToHostDocumentPosition(csharpDocument, absoluteCSharpStartIndex, out var _1, out var hostDocumentIndex) &&
-                    _documentMappingService.TryMapToHostDocumentPosition(csharpDocument, absoluteCSharpStartIndex + length, out var _2, out var _3))
+                if (_documentMappingService.TryMapToHostDocumentPosition(csharpDocument, absoluteCSharpStartIndex, out var _1, out var hostDocumentStartIndex) &&
+                    _documentMappingService.TryMapToHostDocumentPosition(csharpDocument, absoluteCSharpStartIndex + length, out var _2, out var hostDocumentEndIndex))
                 {
-                    ranges.Add(new(kind, hostDocumentIndex, length));
+                    var mappedLength = hostDocumentEndIndex - hostDocumentStartIndex;
+                    if (mappedLength > 0)
+                    {
+                        ranges.Add(new(kind, hostDocumentStartIndex, mappedLength));
+                    }
                 }
 
                 absoluteCSharpStartIndex += length;