On Wed, 14 Sep 2022 18:44:14 GMT, Marius Hanl <mh...@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

Reply via email to