<Swing Dev> Integrated: 8259522: Apply java.io.Serial annotations in java.desktop
Sergey Bylokhov
serb at openjdk.java.net
Fri Jan 15 00:31:01 UTC 2021
On Mon, 11 Jan 2021 06:21:52 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:
> Please review the application of @java.io.Serial annotation (JDK-8202385) to types in the desktop module to enable stricter compile-time checking of serialization-related declarations.
>
> This annotation can be applied to these methods in the module:
>
> private void writeObject(java.io.ObjectOutputStream stream) throws IOException
> private void readObject(java.io.ObjectInputStream stream) throws IOException, ClassNotFoundException
> private void readObjectNoData() throws ObjectStreamException
> ANY-ACCESS-MODIFIER Object writeReplace() throws ObjectStreamException
> ANY-ACCESS-MODIFIER Object readResolve() throws ObjectStreamException
> private static final ObjectStreamField[] serialPersistentFields
> private static final long serialVersionUID
>
> Notes:
> - I have tried to update the comments for serialVersionUID as accurately as possible, but mostly based on the source code history and bugs in JBS where that field was added
> - Some of the readObject/writeObject methods in the javax.swing package does not have a spec, because this package and some others are excluded from the serialization specification.
>
> A similar fix was implemented for java.base module as well:
> http://mail.openjdk.java.net/pipermail/core-libs-dev/2019-August/062046.html
This pull request has now been integrated.
Changeset: 978bed6c
Author: Sergey Bylokhov <serb at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/978bed6c
Stats: 3424 lines in 343 files changed: 2264 ins; 287 del; 873 mod
8259522: Apply java.io.Serial annotations in java.desktop
Reviewed-by: aivanov, psadhukhan
-------------
PR: https://git.openjdk.java.net/jdk/pull/2020
More information about the swing-dev
mailing list