Fix off-by-one in range inclusion (#817)

chrismwendt · web-flow · commit 71eb52f66ed0 · 2022-07-15T13:52:11.000-06:00
diff --git a/template/src/util/api.ts b/template/src/util/api.ts
@@ -682,7 +682,7 @@ const isInRange = (position: sourcegraph.Position, range: Range): boolean => {
     if (position.character < range.column) {
         return false
     }
-    if (position.character > range.column + range.length) {
+    if (position.character >= range.column + range.length) {
         return false
     }
     return true

Original file line number	Diff line number	Diff line change
`@@ -682,7 +682,7 @@ const isInRange = (position: sourcegraph.Position, range: Range): boolean => {`
`682`	`682`	`if (position.character < range.column) {`
`683`	`683`	`return false`
`684`	`684`	`}`
`685`		`- if (position.character > range.column + range.length) {`
	`685`	`+ if (position.character >= range.column + range.length) {`
`686`	`686`	`return false`
`687`	`687`	`}`
`688`	`688`	`return true`