RFR: 8339329: ConstantPoolBuilder#constantValueEntry method doc typo and clarifications
Chen Liang
liach at openjdk.org
Fri Aug 30 17:22:19 UTC 2024
On Fri, 30 Aug 2024 15:08:20 GMT, David M. Lloyd <duke at openjdk.org> wrote:
> Please review this small documentation change to ConstantPoolBuilder to fix a typo and clarify the usage of the `constantValueEntry` method.
src/java.base/share/classes/java/lang/classfile/constantpool/ConstantPoolBuilder.java line 479:
> 477: * {@return A {@link ConstantValueEntry} describing the provided
> 478: * Integer, Long, Float, Double, or String constant}
> 479: * The returned consant is suitable as a value for the
This information is already available on `ConstantValueEntry`. Since we already link to it on the return type, I feel like this is a redundancy.
However, `ConstantValueEntry::constantValue()` is missing a link to this API; would be nice if you can add that. And `ConstantValueEntry` is missing a link to `Attributes::constantValue()` too.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20796#discussion_r1739164493
More information about the core-libs-dev
mailing list