Add debug info to invalid offset error

Veykril · Veykril · commit 955989431270 · 2024-06-05T10:20:05.000+02:00
diff --git a/src/tools/rust-analyzer/crates/rust-analyzer/src/lsp/from_proto.rs b/src/tools/rust-analyzer/crates/rust-analyzer/src/lsp/from_proto.rs
@@ -37,8 +37,9 @@ pub(crate) fn offset(
                 .ok_or_else(|| format_err!("Invalid wide col offset"))?
         }
     };
-    let text_size =
-        line_index.index.offset(line_col).ok_or_else(|| format_err!("Invalid offset"))?;
+    let text_size = line_index.index.offset(line_col).ok_or_else(|| {
+        format_err!("Invalid offset {line_col:?} (line index length: {:?})", line_index.index.len())
+    })?;
     Ok(text_size)
 }
 

Original file line number	Diff line number	Diff line change
`@@ -37,8 +37,9 @@ pub(crate) fn offset(`
`37`	`37`	`.ok_or_else(\|\| format_err!("Invalid wide col offset"))?`
`38`	`38`	`}`
`39`	`39`	`};`
`40`		`- let text_size =`
`41`		`- line_index.index.offset(line_col).ok_or_else(\|\| format_err!("Invalid offset"))?;`
	`40`	`+ let text_size = line_index.index.offset(line_col).ok_or_else(\|\| {`
	`41`	`+ format_err!("Invalid offset {line_col:?} (line index length: {:?})", line_index.index.len())`
	`42`	`+ })?;`
`42`	`43`	`Ok(text_size)`
`43`	`44`	`}`
`44`	`45`