On Mon, 27 Sep 2021 18:40:10 GMT, Joe Darcy <darcy@openjdk.org> wrote:
Follow-up change to JDK-8232230, augmentations to javac's Xlint:serial checking are out for review (https://github.com/openjdk/jdk/pull/5709) and java.util.concurrent would need some changes to pass under the expanded checks.
The changes are to suppress warnings where non-transient fields in serializable types are not declared with a type statically known to be serializable. That isn't necessarily a correctness issues, but it does merit further scrutiny.
In ForkJoinTask, the SuppressWarnings annotation previously applied in JDK-8232230, seems to have been misplaced to a different field; this change corrects it.
This pull request has now been integrated. Changeset: 2072bc77 Author: Joe Darcy <darcy@openjdk.org> URL: https://git.openjdk.java.net/jdk/commit/2072bc77b4541c283adaa7eb51a38adcaf71... Stats: 3 lines in 2 files changed: 2 ins; 1 del; 0 mod 8274391: Suppress more warnings on non-serializable non-transient instance fields in java.util.concurrent Reviewed-by: lancea, bpb, alanb, iris ------------- PR: https://git.openjdk.java.net/jdk/pull/5718