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

[Adam Sotona]

On Thu, 10 Oct 2024 19:13:01 GMT, Chen Liang <liach at openjdk.org> wrote:

> 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.

src/java.base/share/classes/java/lang/classfile/instruction/DiscontinuedInstruction.java line 37:

> 35: import jdk.internal.javac.PreviewFeature;
> 36: 
> 37: import static java.util.Objects.requireNonNull;

Isn't it unused?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21458#discussion_r1797148194