Просмотр исходного кода

Merge pull request #19639 from overleaf/jdt-line-tracker-max-callstack-event

Adding events to line-tracker

GitOrigin-RevId: 97601af3c332244d10a3e97eb22a80a0eb240ad3
Jimmy Domagala-Tang 2 лет назад
Родитель
Сommit
91125f8b1c

+ 13 - 2
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
+              }
             }
           }
         )