Fix bug in origin location computation.

KarelPeeters · KarelPeeters · commit d8c868f7d92b · 2024-10-14T00:39:16.000+02:00
In particular the second "&lt;=" is wrong, it should be "&lt;", since ranges are end-exclusive. For clarity just replace it with Range::contains.
diff --git a/src/renderer/display_list.rs b/src/renderer/display_list.rs
@@ -1149,7 +1149,7 @@ fn format_header<'a>(
                 ..
             } = item
             {
-                if main_range >= range.0 && main_range <= range.1 + end_line.len() {
+                if (range.0..range.1 + end_line.len()).contains(&main_range) {
                     let char_column = text[0..(main_range - range.0).min(text.len())]
                         .chars()
                         .count();

Original file line number	Diff line number	Diff line change
`@@ -1149,7 +1149,7 @@ fn format_header<'a>(`
`1149`	`1149`	`..`
`1150`	`1150`	`} = item`
`1151`	`1151`	`{`
`1152`		`- if main_range >= range.0 && main_range <= range.1 + end_line.len() {`
	`1152`	`+ if (range.0..range.1 + end_line.len()).contains(&main_range) {`
`1153`	`1153`	`let char_column = text[0..(main_range - range.0).min(text.len())]`
`1154`	`1154`	`.chars()`
`1155`	`1155`	`.count();`