RFR: JDK-8223725: j.l.c.MethodHandleDesc::of throws undocumented exception IllegalArgumentException

Vicente Romero vicente.romero at oracle.com
Mon May 20 19:58:55 UTC 2019


thanks for the review,
Vicente

On 5/20/19 2:09 PM, Roger Riggs wrote:
> Hi Vicente,
>
> Looks ok.
>
> Can you ident the 2nd and subsequent lines of the javadoc tag,
> as is done in the other tags of the method.
> No need for another webrev.
>
> Thanks, Roger
>
> On 05/20/2019 01:07 PM, Vicente Romero wrote:
>> 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