jmx-dev RFR: 8274398: Suppress more warnings on non-serializable non-transient instance fields in management libs [v2]

Joe Darcy darcy at openjdk.java.net
Thu Sep 30 17:00:24 UTC 2021


> Follow-up change to JDK-8232442, augmentations to javac's Xlint:serial checking are out for review (#5709) and various management libraries 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.
> 
> I'll run a script to update copyright years before a push.

Joe Darcy has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:

 - Add comments and update copyrights.
 - Merge branch 'master' into JDK-8274398
 - 8274398: Suppress more warnings on non-serializable non-transient instance fields in management libs

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/5726/files
  - new: https://git.openjdk.java.net/jdk/pull/5726/files/bc8dde4f..a6f37fa5

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=5726&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=5726&range=00-01

  Stats: 5574 lines in 212 files changed: 3989 ins; 1022 del; 563 mod
  Patch: https://git.openjdk.java.net/jdk/pull/5726.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/5726/head:pull/5726

PR: https://git.openjdk.java.net/jdk/pull/5726


More information about the jmx-dev mailing list