> 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.
Chen Liang has updated the pull request incrementally with one additional commit since the last revision: asotona review remarks ------------- Changes: - all: https://git.openjdk.org/jdk/pull/21458/files - new: https://git.openjdk.org/jdk/pull/21458/files/79443792..b4f0ebcc Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=21458&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=21458&range=00-01 Stats: 940 lines in 3 files changed: 1 ins; 939 del; 0 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
