RFR: 8309819: Clarify API note in Class::getName and MethodType::toMethodDescriptorString [v4]
Chen Liang
liach at openjdk.org
Wed Jun 14 07:15:09 UTC 2023
On Wed, 14 Jun 2023 06:03:48 GMT, Alan Bateman <alanb at openjdk.org> wrote:
>> Chen Liang has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Convert the note in fromDescriptorString to apiNote
>
> src/java.base/share/classes/java/lang/Class.java line 928:
>
>> 926: * @apiNote
>> 927: * This is not a strict inverse of {@link #forName(String) forName}.
>> 928: * For normal classes, distinct classes which share a common name but
>
> Readers may wonder what "normal classes" means here, it might be simpler to just drop introducing that term and just use the structure from the note removed from descriptorString.
This might wish to refer to "classes that can be described nominally", but same might be said for hidden classes as well. So your suggestion is better.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14411#discussion_r1229124490
More information about the core-libs-dev
mailing list