RFR: 8279214: Memory leak in Scene after dragging a cell
Ambarish Rapte
arapte at openjdk.org
Tue Nov 22 09:36:31 UTC 2022
On Wed, 14 Sep 2022 18:44:14 GMT, Marius Hanl <mhanl at openjdk.org> wrote:
> This PR fixes a memory leak in dnd code in the `MouseHandler` of a `Scene`.
> The memory leak occurs after calling `startFullDrag()`. The `fullPDRTmpTargetWrapper` is then populated but never cleared.
> Fix is to call `clear()` on the `fullPDRTmpTargetWrapper`, very similar to https://github.com/openjdk/jfx/pull/448 ([JDK-8264330](https://bugs.openjdk.java.net/browse/JDK-8264330)).
Looks good to me too.
`fullPDRTmpTargetWrapper` gets cleared both before and after gesture.
-------------
Marked as reviewed by arapte (Reviewer).
PR: https://git.openjdk.org/jfx/pull/899
More information about the openjfx-dev
mailing list