RFR: JDK-8223725: j.l.c.MethodHandleDesc::of throws undocumented exception IllegalArgumentException
Vicente Romero
vicente.romero at oracle.com
Mon May 20 17:07:21 UTC 2019
how does it look now?
Thanks,
Vicente
On 5/17/19 3:58 PM, Vicente Romero wrote:
> Hi Roger,
>
> Thanks for the review, my apologies, I made a mistake an applied the
> change to the wrong method. I have corrected the patch [1] and the CSR
> [2]. I have updated the language used as you suggested,
>
> Thanks,
> Vicente
>
> [1] http://cr.openjdk.java.net/~vromero/8223725/webrev.01/
> [2] https://bugs.openjdk.java.net/browse/JDK-8223920
>
> On 5/17/19 2:05 PM, Roger Riggs wrote:
>> Hi Vicente,
>>
>> What's the difference in the IAE exceptions describing IAE's as
>> being "incorrect formats' vs 'is invalid' or 'not valid'?
>>
>> Each requires a spec for valid and invalid strings or is undefined.
>>
>> Can the new IAE use the same language as is used for the ofDescriptor
>> method?
>> Avoiding any question about differences.
>>
>> 69: Also, is the word 'lookup' overloaded? Is 'lookup' is
>> significant in this context?
>>
>> Roger
>>
>>
>> On 05/14/2019 06:37 PM, Vicente Romero wrote:
>>> Please review fix [1] for [2] and the corresponding CSR at [3]. The
>>> fix is just adding a missing @throws at method
>>> j.l.c.MethodHandleDesc::of. It is a one liner fix,
>>>
>>> Thanks,
>>> Vicente
>>>
>>> [1] http://cr.openjdk.java.net/~vromero/8223725/webrev.00/
>>> [2] https://bugs.openjdk.java.net/browse/JDK-8223725
>>> [3] https://bugs.openjdk.java.net/browse/JDK-8223920
>>
>
More information about the core-libs-dev
mailing list