RFR: 8252811: The list of cells in a VirtualFlow is cleared every time the number of items changes
Jose Pereda
jpereda at openjdk.java.net
Tue Sep 8 12:01:21 UTC 2020
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.
-------------
Commit messages:
- Add test to verify the sheet children size remains constant when adding or removing new items
- Remove old fix RT-13965
Changes: https://git.openjdk.java.net/jfx/pull/298/files
Webrev: https://webrevs.openjdk.java.net/?repo=jfx&pr=298&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8252811
Stats: 38 lines in 2 files changed: 32 ins; 6 del; 0 mod
Patch: https://git.openjdk.java.net/jfx/pull/298.diff
Fetch: git fetch https://git.openjdk.java.net/jfx pull/298/head:pull/298
PR: https://git.openjdk.java.net/jfx/pull/298
More information about the openjfx-dev
mailing list