On Tue, 23 Nov 2021 11:00:51 GMT, Jose Pereda <jper...@openjdk.org> wrote:

> This PR fixes a memory leak, as a follow-up of 
> [JDK-8241840](https://bugs.openjdk.java.net/browse/JDK-8241840).
> 
> The tests pass on desktop (Linux/macOS/Windows) and there is no regression on 
> Android (see [JDK-8254605](https://bugs.openjdk.java.net/browse/JDK-8254605)).

This was my mistake -- well, actually, an unfortunate design choice by gradle, 
but one I'm so familiar with I'm surprised I got bit by it. gradle will not 
rerun a test if it thinks the tests are up-to-date, and that's exactly what 
happened to me. Twice. This is why when I run individual tests, I (almost) 
always add the `cleanTest` target. When I do that, it fails for me, too, 
without your fix.

-------------

Marked as reviewed by kcr (Lead).

PR: https://git.openjdk.java.net/jfx/pull/677

Reply via email to