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