On Tue, 29 Jul 2025 22:36:09 GMT, Jeremy Wood <[email protected]> wrote:
>> Prasanta Sadhukhan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Use named property listener, update test > > src/java.desktop/share/classes/javax/swing/JPopupMenu.java line 1427: > >> 1425: indexCounter++; >> 1426: } >> 1427: } > > This may be a naive question: is the lambda we use for `propListener` > Serializable? > > (If so: great. If not: won't we get inconsistent results if we're working > with a deserialized copy of a JPopupMenu? The original will have a non-null > `propListener` field, and the copy will have a null `propListener`? Or am I > missing something?) It is transient so its value should not be included in the default serialization process in my opinion ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/26407#discussion_r2241461858
