On Tue, 8 Sep 2020 11:56:51 GMT, Jose Pereda <jper...@openjdk.org> wrote:
> This PR removes an old fix (RT-13965/JDK-8113318), which was applied in 2011 > to avoid a memory leak in > `VirtualFlow::sheetChildren`, after new items were constantly added. > With the current VirtualFlow implementation, there are in place the necessary > methods that take care of cleaning or > adding new cells to the sheetChildren list, and such memory leak doesn't > exist, the size remains constant when new > items are added or removed (see included unity test). The call to > `sheetchildren.clear()`, on the contrary, has a big > impact in performance and CPU usage, when new items are constantly added, as > has been reported in JDK-8185886/#108. This pull request has now been integrated. Changeset: d10f948e Author: Jose Pereda <jper...@openjdk.org> URL: https://git.openjdk.java.net/jfx/commit/d10f948e Stats: 38 lines in 2 files changed: 6 ins; 32 del; 0 mod 8252811: The list of cells in a VirtualFlow is cleared every time the number of items changes Reviewed-by: aghaisas, kcr ------------- PR: https://git.openjdk.java.net/jfx/pull/298
