RFR: 8304139: Add <init> and <clinit> method constants to ConstantDescs [v3]

Chen Liang liach at openjdk.org
Fri Mar 17 00:49:32 UTC 2023


On Thu, 16 Mar 2023 20:43:30 GMT, Mandy Chung <mchung at openjdk.org> wrote:

>> Chen Liang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove main related constants, update specification for initialization constants
>
> src/java.base/share/classes/java/lang/constant/ConstantDescs.java line 315:
> 
>> 313:      * @since 21
>> 314:      */
>> 315:     public static final MethodTypeDesc MTD_void = MethodTypeDesc.of(CD_void);
> 
> This is a generic method type descriptor and not restrict to initialization methods.   So I think no need to mention that "This is the method type of initialization methods ..." and  `@jvms, @see and @jls" references.
> 
> Maybe it can be as simple as:
> 
> 
> 
> Nominal descriptor representing the method descriptor {@code ()V},
> taking no argument and returning {@code void}.

Done. Also updated issue description after removing main method constants.

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

PR: https://git.openjdk.org/jdk/pull/13020


More information about the core-libs-dev mailing list