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!

best regards,

-- daniel

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



Reply via email to