On Tue, 13 Jun 2023 21:54:10 GMT, Mandy Chung <mch...@openjdk.org> wrote:
>> Chen Liang has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Update src/java.base/share/classes/java/lang/Class.java
>>
>> Co-authored-by: Mandy Chung <mandy.ch...@oracle.com>
>
> src/java.base/share/classes/java/lang/invoke/MethodType.java line 1233:
>
>> 1231: * will appear identical when viewed within descriptor strings.
>> 1232: * In addition, the result string is not a bytecode descriptor if
>> this
>> 1233: * method type cannot be {@linkplain ##descriptor described
>> nominally}.
>
> This method is equivalent to `MethodType::descriptorString` which specifies
> the result descriptor string if a method type descriptor not. So I think
> this sentence may not be needed.
Should we consolidate this distinct class loader note into `descriptorString`
too?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14411#discussion_r1228827595
