RFR: 8230310: SocksSocketImpl should handle the IllegalArgumentException thrown by ProxySelector.select usage

[Jaikiran Pai]

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
>>
>>
>