Skip to content

Commit 2f41508

Browse files
fix(turtles): Clear intervals when removing turtles to prevent memory leaks (sugarlabs#5106)
1 parent 90aa31e commit 2f41508

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

js/turtles.js

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -467,6 +467,15 @@ Turtles.TurtlesModel = class {
467467
*/
468468
removeTurtle(index) {
469469
if (index >= 0 && index < this._turtleList.length) {
470+
const turtle = this._turtleList[index];
471+
472+
// Clear any active intervals to prevent memory leaks
473+
if (turtle.interval !== undefined) {
474+
const intervalId = turtle.interval;
475+
clearInterval(intervalId);
476+
turtle.interval = undefined;
477+
}
478+
470479
this._turtleList.splice(index, 1);
471480
}
472481
}

0 commit comments

Comments
 (0)