Re: RFR: JDK-8223803: j.l.c.MethodTypeDesc::insertParameterTypes​ doesn't control type of parameters

Vicente Romero vicente.romero at oracle.com
Thu May 16 22:39:37 UTC 2019


Hi Roger,

Thanks for the review, I will do the re-wrapping before pushing,

Thanks,
Vicente

On 5/16/19 5:15 PM, Roger Riggs wrote:
> Hi Vicente,
>
> Looks ok.
> And the CSR too.
>
> Please re-wrap the lines to 80 chars to make diffs easier to read.
> ConstantDesc.java and MethodTypeDesc.java
>
> Thanks, Roger
>
> On 05/15/2019 05:27 PM, Vicente Romero wrote:
>> Please review fix for [1] at [2] and the related CSR at [3]. Method 
>> java.lang.constant.MethodTypeDesc::insertParameterTypes​ should 
>> control the type of parameters being inserted. In particular, no 
>> parameter can be a class descriptor of primitive type |void|. This 
>> fix is updating the spec of the mentioned method plus it is adding a 
>> test to make sure that the implementation is in sync with the 
>> specification. As mentioned in the CSR and apiNote has been added to 
>> method j.l.c.ConstantDesc::resolveConstantDesc to clarify that a 
>> method type descriptor can have more than 255 argument slots so at 
>> resolution time, as a j.l.i.MethodType, errors can be thrown.
>>
>> Thanks,
>> Vicente
>>
>> [1] https://bugs.openjdk.java.net/browse/JDK-8223803
>> [2] http://cr.openjdk.java.net/~vromero/8223803/webrev.00/
>> [3] https://bugs.openjdk.java.net/browse/JDK-8223997
>



More information about the core-libs-dev mailing list