RFR: 8333749: Consolidate ConstantDesc conversion in java.base [v2]

Chen Liang liach at openjdk.org
Thu Jun 6 21:56:15 UTC 2024


On Thu, 6 Jun 2024 19:30:21 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

>> Chen Liang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   mt -> md (desc)
>
> src/java.base/share/classes/jdk/internal/constant/ConstantUtils.java line 76:
> 
>> 74:      * type and parameter types can be described nominally.
>> 75:      */
>> 76:     public static MethodTypeDesc methodDesc(MethodType type) {
> 
> Please name these methods `methodTypeDesc`, since we also have the `Method` type.

Thanks for the suggestion. Done.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19585#discussion_r1630306158


More information about the core-libs-dev mailing list