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