RFR: 8329099: Undocumented exception thrown by Instruction factory methods accepting Opcode

Adam Sotona asotona at openjdk.org
Tue Apr 2 09:43:11 UTC 2024


`IllegalArgumentException` thrown by some static factory methods of the following `java.lang.classfile.instruction` interfaces are not documented:

- `ArrayLoadInstruction`
- `ArrayStoreInstruction`
- `BranchInstruction`
- `ConvertInstruction`
- `DiscontinuedInstruction`
- `FieldInstruction`
- `InvokeInstruction`
- `LoadInstruction`
- `MonitorInstruction`
- `NewPrimitiveArrayInstruction`
- `OperatorInstruction`
- `ReturnInstruction`
- `StackInstruction`
- `StoreInstruction`
- `TypeCheckInstruction`

`NewPrimitiveArrayInstruction::of` factory method also does not perform the check for invalid argument.

This patch adds all the missing `@throws` Javadoc tags and fixes `NewPrimitiveArrayInstruction::of` factory method to perform the argument check.

Please review.

Thank you,
Adam

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

Commit messages:
 - 8329099: Undocumented exception thrown by Instruction factory methods accepting Opcode

Changes: https://git.openjdk.org/jdk/pull/18578/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=18578&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8329099
  Stats: 36 lines in 15 files changed: 36 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/18578.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/18578/head:pull/18578

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


More information about the core-libs-dev mailing list