On Sat, 30 Oct 2021 10:56:40 GMT, Florian Kirmaier <fkirma...@openjdk.org> 
wrote:

> This fixes the new ControlAcceleratorBug which was Introduced in JavaFX17.
> To fix it, I've made the Value of the WeakHashMap also weak. 
> We only keep this value to remove it as a listener later on. Therefore there 
> shouldn't be issues by making this value weak.
> 
> 
> I've seen this Bug very very often, in the last weeks. Most of the 
> applications I've seen are somehow affected by this bug.
> 
> It basically **breaks every application with menu bars and multiple stages** 
> - which is the majority of enterprise applications. It's especially annoying 
> because it takes some time until the application gets unstable.
> 
> Therefore **I would recommend** after this fix is approved, **to make a new 
> version for JavaFX17** with this fix because this bug is so severe.

>   * [JDK-8274022](https://bugs.openjdk.java.net/browse/JDK-8274022): 
> Additional Memory Leak in ControlAcceleratorSupport ⚠️ Title mismatch between 
> PR and JBS.

@FlorianKirmaier as a reminder, the PR title must match the JBS title. Note 
that I modified the JBS title slightly, so when you do fix this, you will need 
to grab the updated title:


8274022: Additional Memory Leak in ControlAcceleratorSupport

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

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

Reply via email to