RFR 8151586: Wrong exception catch for FTPClient in JDK-8055032

[vyom]

Hi Chris,

thanks for review, please find the updated webrev. I updated the 
existing test case to cover this issue.

http://cr.openjdk.java.net/~vtewari/8151586/webrev0.1/index.html 
<http://cr.openjdk.java.net/%7Evtewari/8151586/webrev0.1/index.html>

Thanks,
Vyom

On Tuesday 15 March 2016 05:08 PM, Chris Hegarty wrote:
> Vyom,
>
> On 15/03/16 09:51, vyom wrote:
>> Hi,
>> Please review the below fix.
>>
>> Bug: JDK-8151586 : Wrong exception catch for FTPClient in JDK-8055032
>> Webrev :http://cr.openjdk.java.net/~rgoel/~vyom/8151586/webrev0.0/
>> <http://cr.openjdk.java.net/%7Ergoel/%7Evyom/8151586/webrev0.0/>
>
> The source change looks ok.
>
> The test attempts to connect to an external ftp site. It
> just fails on my machine with java.net.NoRouteToHostException.
>
> Does the test need to run in othervm mode. I think not.
>
> To simplify the test, just add 'throws IOException' to
> main, so you don't have to deal with wrapping these in
> RuntimeException. If a test fails with an IOException it
> will be reported as failed by the test harness. Try it.
>
> -Chris.