Fix out of bounds cursor checking
This commit is contained in:
parent
7c081e8939
commit
438f0b4d97
1 changed files with 7 additions and 2 deletions
|
|
@ -14,13 +14,18 @@ export function positionToLineAndColumn(
|
||||||
throw new Error("Position cannot be negative");
|
throw new Error("Position cannot be negative");
|
||||||
}
|
}
|
||||||
|
|
||||||
if (position > text.length) {
|
text = text.replace("\r", "");
|
||||||
|
|
||||||
|
if (
|
||||||
|
position >
|
||||||
|
text.length + 1
|
||||||
|
// +1 to account for the cursor being after last character
|
||||||
|
) {
|
||||||
throw new Error(
|
throw new Error(
|
||||||
`Position ${position} exceeds text length ${text.length}`
|
`Position ${position} exceeds text length ${text.length}`
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
text = text.replace("\r", "");
|
|
||||||
const textUpToPosition = text.substring(0, position);
|
const textUpToPosition = text.substring(0, position);
|
||||||
const lines = textUpToPosition.split("\n");
|
const lines = textUpToPosition.split("\n");
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue