On Tue, 29 Jul 2025 11:53:05 GMT, Prasanta Sadhukhan <[email protected]>
wrote:
>> src/java.desktop/share/classes/javax/swing/JPopupMenu.java line 1383:
>>
>>> 1381: values.addElement("propListener");
>>> 1382: values.addElement(propListener);
>>> 1383: }
>>
>> Do we need to serialise `propListener`? What is the value in serialising the
>> listener?
>>
>> Will `invoker` serialise `propListener`? If it does, then we need to
>> serialise it to be able to remove the listener after the components are
>> deserialised and the `invoker` is changed.
>
> There was failure in CI in some test citing NotSerializableException which
> was solved by serialising propListener
Weird… transient fields aren't serialised.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26407#discussion_r2239590058