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