From fa12d56552f7f5b6ad57c6061f12a8ad448c2fb6 Mon Sep 17 00:00:00 2001
From: Iron-E <36409591+Iron-E@users.noreply.github.com>
Date: Sat, 4 Jul 2020 15:59:46 +0000
Subject: [PATCH] Fix going out of bounds in a buffer

`strict_indexing` was set to `true` in the `nvim_buf_get_lines` call, so when `row` got to the end of the buffer it would cause an error.

Setting `strict_indexing` to `false` caused `line` to be `nil` sometimes, so I added some logic to handle that.
---
 lua/diagnostic/util.lua | 27 +++++++++++++++------------
 1 file changed, 15 insertions(+), 12 deletions(-)

diff --git a/lua/diagnostic/util.lua b/lua/diagnostic/util.lua
index 41f2f12..5904e42 100644
--- a/lua/diagnostic/util.lua
+++ b/lua/diagnostic/util.lua
@@ -146,19 +146,22 @@ function M.locations_to_items(locations)
   for _, temp in ipairs(rows) do
     local pos = temp.start
     local row = pos.line
-    local line = api.nvim_buf_get_lines(0, row, row+1, true)[1]
-    local col
-    if pos.character > #line then
-      col = #line
-    else
-      col = vim.str_byteindex(line, pos.character)
+    local line = api.nvim_buf_get_lines(0, row, row+1, false)[1]
+    if line then
+      local col
+      if pos.character > #line then
+        col = #line
+      else
+        col = vim.str_byteindex(line, pos.character)
+      end
+
+      table.insert(items, {
+        bufnr = bufnr,
+        lnum = row + 1,
+        col = col + 1;
+        text = line
+      })
     end
-    table.insert(items, {
-      bufnr = bufnr,
-      lnum = row + 1,
-      col = col + 1;
-      text = line
-    })
   end
   return items
 end