On Mon, 4 Mar 2024 11:07:07 GMT, Johan Vos <j...@openjdk.org> wrote:

> Explicitly remove old global ref value from menu->jCallback.
> This avoids the possibility of that value (which can be reused for other 
> refs) to be deleted twice.

This is a safe fix that is clearly the right thing to do.

I recommend changing the Bug title (and then the PR title to match) to add 
"macOS" and "GlassMenu" somewhere in the title to make it a little more 
informative, but I'll leave it up to you.

This is a simple enough fix that a single reviewer is sufficient. Once 
integrated, I recommend backporting this to jfx22u for 22.0.1 (and possibly to 
earlier releases as you see fit).

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

Marked as reviewed by kcr (Lead).

PR Review: https://git.openjdk.org/jfx/pull/1388#pullrequestreview-1914377783

Reply via email to