Merge pull request #72 from alexnask/master

Check that the goto definition cursor is within bounds
This commit is contained in:
Auguste Rame 2020-05-21 14:03:55 -04:00 committed by GitHub
commit ef7dea8104
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -283,6 +283,7 @@ fn nodeToCompletion(list: *std.ArrayList(types.CompletionItem), analysis_ctx: *D
}
fn identifierFromPosition(pos_index: usize, handle: DocumentStore.Handle) []const u8 {
if (pos_index + 1 >= handle.document.text.len) return &[0]u8{};
var start_idx = pos_index;
while (start_idx > 0 and
@ -290,7 +291,7 @@ fn identifierFromPosition(pos_index: usize, handle: DocumentStore.Handle) []cons
{}
var end_idx = pos_index;
while (end_idx < handle.document.text.len and
while (end_idx < handle.document.text.len - 1 and
(std.ascii.isAlNum(handle.document.text[end_idx]) or handle.document.text[end_idx] == '_')) : (end_idx += 1)
{}