motiontrail: fix crash on invalid selection idx

jgmdev · jgmdev · commit 895ce2df974c · 2026-02-24T18:43:36.000-04:00
diff --git a/manifest.json b/manifest.json
@@ -2002,7 +2002,7 @@
       "id": "motiontrail",
       "mod_version": "3.1",
       "path": "plugins/motiontrail.lua",
-      "version": "0.3"
+      "version": "0.4"
     },
     {
       "description": "Allows moving back and forward between document positions, reducing the amount of scrolling",
diff --git a/plugins/motiontrail.lua b/plugins/motiontrail.lua
@@ -39,11 +39,13 @@ core.add_thread(function()
 end)
 
 local function get_caret_size(dv, i)
-  local line, col = dv.doc:get_selection_idx(i)
-  local chw = dv:get_font():get_width(dv.doc:get_char(line, col))
   local w = style.caret_width
   local h = dv:get_line_height()
 
+  local line, col = dv.doc:get_selection_idx(i)
+  if not line or not col then return w, h end
+  local chw = dv:get_font():get_width(dv.doc:get_char(line, col))
+
   if dv.doc.overwrite then
     w = chw
     h = style.caret_width * 2

Original file line number	Diff line number	Diff line change
`@@ -2002,7 +2002,7 @@`
`2002`	`2002`	`"id": "motiontrail",`
`2003`	`2003`	`"mod_version": "3.1",`
`2004`	`2004`	`"path": "plugins/motiontrail.lua",`
`2005`		`- "version": "0.3"`
	`2005`	`+ "version": "0.4"`
`2006`	`2006`	`},`
`2007`	`2007`	`{`
`2008`	`2008`	`"description": "Allows moving back and forward between document positions, reducing the amount of scrolling",`