RFR: 8317356: Test ClassFile API if it deals with nulls correctly across the whole API

Chen Liang liach at openjdk.org
Thu Oct 10 19:17:24 UTC 2024


Please review this patch that adds a test by @nizarbenalla to perform null checks across the ClassFile API. This is an updated version of #20556 that minimizes impact on our implementation code.

Notes:
1. There's one change in `MethodHandleProxies` to explicitly use platform class loader instead of `null` for boot class loader. Tests work fine.
2. The null check test uses the same set of classes as CorpusTest to make it suitable for tier 1.
3. The test is not comprehensive; currently it is permissive toward IAE, because sometimes some substitution arguments are invalid, especially primitive ClassDesc or Opcode with wrong types.

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

Commit messages:
 - 8317356: Test ClassFile API if it deals with nulls correctly across the whole API

Changes: https://git.openjdk.org/jdk/pull/21458/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=21458&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8317356
  Stats: 1202 lines in 38 files changed: 1085 ins; 14 del; 103 mod
  Patch: https://git.openjdk.org/jdk/pull/21458.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/21458/head:pull/21458

PR: https://git.openjdk.org/jdk/pull/21458


More information about the core-libs-dev mailing list