RFR: 8283551: ControlAcceleratorSupport menu items listener causes memory leak [v2]

[Kevin Rushforth]

On Fri, 24 Feb 2023 19:02:25 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> In general should we use new syntax like that instanceof syntax? JavaFX requires Java 17 now, but if changes were to be backported then using the new syntax makes it a bit more difficult.
>
> that will be "SEP" (someone else's problem) ;-)
> 
> fx requires jdk17 as a boot jdk per JDK-8276144

Right. For newly added or modified code, it's fine to use JDK 17 language constructs and API, unless you know that the code will be backported right away. In particular, we have been using pattern-matching `instanceof` in other places.

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

PR: https://git.openjdk.org/jfx/pull/1037