RFR: 8208088: Memory Leak in ControlAcceleratorSupport [v6]

[Daniel Peintner]

Hi,

I don't wanna clutter the list but I noticed this issue also and can point
you to a simple test (adding MenuButton is sufficient, no need for any
MenuItem).

See test here:
https://github.com/Sandec/JMemoryBuddy/issues/5#issuecomment-911617849

The test provided fails on JavaFX (v17) and succeeds on v16.
@FlorianKirmaier mentioned that for him the test on Mac works also for
JavaFX17. Maybe it depends on the operating system also...

Note: The test should be usable "as is" since it uses JMemoryBuddy which is
already part of the JFX repo.
(some minor tweaks might be necessary due to the minor assert differences
in Junit4 vs. Junit5)

Hope this helps,

-- Daniel


On Fri, Sep 10, 2021 at 1:40 PM Kevin Rushforth <kcr at openjdk.java.net>
wrote:

> On Fri, 10 Sep 2021 11:28:25 GMT, Marius Hanl <mhanl at openjdk.org> wrote:
>
> > Could you please report the bug at https://bugreport.java.com/ with a
> minimum reproducible example (written in Java)?
>
> I was just going to mention the same thing. See the project [README](
> https://github.com/openjdk/jfx#readme).
>
> -------------
>
> PR: https://git.openjdk.java.net/jfx/pull/429
>