From f2be5b3f7455f8011b99187c8353d361c5a7ffe0 Mon Sep 17 00:00:00 2001 From: Jimmy Domagala-Tang Date: Tue, 30 Jul 2024 13:14:56 -0400 Subject: [PATCH] Merge pull request #19639 from overleaf/jdt-line-tracker-max-callstack-event Adding events to line-tracker GitOrigin-RevId: 97601af3c332244d10a3e97eb22a80a0eb240ad3 --- .../extensions/spelling/line-tracker.ts | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/services/web/frontend/js/features/source-editor/extensions/spelling/line-tracker.ts b/services/web/frontend/js/features/source-editor/extensions/spelling/line-tracker.ts index d62c24291c..02711cf827 100644 --- a/services/web/frontend/js/features/source-editor/extensions/spelling/line-tracker.ts +++ b/services/web/frontend/js/features/source-editor/extensions/spelling/line-tracker.ts @@ -111,9 +111,20 @@ export class LineTracker { * |3|yy| * |4|cccc| */ + const changes = new Array(insertedText.lines).fill(true) - this._lines.splice(startLine - 1 + lineShift, 1, ...changes) - lineShift += changes.length - 1 + try { + this._lines.splice(startLine - 1 + lineShift, 1, ...changes) + lineShift += changes.length - 1 + } catch (error) { + if (error instanceof RangeError) { + throw new OError(error.message).withInfo({ + changesSize: changes.length, + }) + } + + throw error + } } } )