On 29/08/19 3:54 PM, Daniel Fuchs wrote:
> Thanks! Looks fine. I see you have used Assert.fail and that's
> better than what I suggested.
>
> I can be your sponsor if you want. Count me as Reviewer!
Yes please. I have updated the webrev at the same location, with just an
update to the Reviewed-by field in the commit log.
Thank you Daniel for reviewing and sponsoring.
-Jaikiran
>
> On 29/08/2019 10:48, Jaikiran Pai wrote:
>>> Maybe simply insert:
>>> throw new AssertionError("expected exception not thrown");
>>> between lines 75 and 76?
>> You are right of course, I missed that. I have now updated the test and
>> the webrev is at the same location
>> http://cr.openjdk.java.net/~jpai/webrev/8230310/2/webrev/
>>
>> -Jaikiran
>>
>>
>
