Memory leak in table fields [message #1778506] |
Fri, 15 December 2017 10:35 |
Simon Meer Messages: 7 Registered: February 2017 |
Junior Member |
|
|
It seems like there may be a memory leak in tables that update often.
In Table.js, there is an array _updatedRows which stores all rows that have been updated during the lifetime of the table field:
scout.TableField.prototype._onTableChanged = function(event) {
[...]
if (event.type === 'rowsUpdated') {
scout.arrays.pushAll(this._updatedRows, event.rows);
return;
}
[...]
};
However, since it's an array, it will keep growing for every row that has been touched in any way. Our table can have up to 200 rows that update every second, resulting in a... quite big heap in a short time.
Suggestion:
Could _updatedRows perhaps be removed in favor of simply setting touched to true?
[Updated on: Fri, 15 December 2017 13:48] Report message to a moderator
|
|
|
|
Powered by
FUDForum. Page generated in 0.03100 seconds