On Tue, 11 Feb 2025 23:33:42 GMT, Andy Goryachev <ango...@openjdk.org> wrote:
>> ## Root Cause >> Animation gets started in a background thread, which causes the animation >> handler to run in the FX application thread, thus creating simultaneous >> access to the control's fields (list of children in this case). >> >> ## Solution >> Postpone the animation unless running in the FX application thread. There >> is no functional difference if the component is created/used in the FX >> application thread. > > Andy Goryachev has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brought in by the merge/rebase. The pull request contains eight additional > commits since the last revision: > > - Merge branch '8349756.pagination.leak' into > 8349105.thread.safety.pagination > - back to constructor > - memory leak > - remove gc > - slow down gc > - gc > - Merge branch 'master' into 8349105.thread.safety.pagination > - postpone animation > For testing purposes, merged this PR with the memory leak fix #1705 . Once > the latter is integrated, unrelated changes will disappear. Can you merge master so this will be realized? ------------- PR Comment: https://git.openjdk.org/jfx/pull/1698#issuecomment-2659987368
