On 08/12/2016 10:56 PM, Ramanand Patil wrote:
> Here is the updated Webrev:
> http://cr.openjdk.java.net/~rpatil/8161016/webrev.01/
I think my version (below) is a little more straightforward, but no
pressure. Your version is okay too, and does the minimal change. Thumbs up.
Thanks,
-Aleksey
> } catch (IOException ioex) {
> if (p == Proxy.NO_PROXY) {
> throw ioex;
> }
> sel.connectFailed(uri, p.address(), ioex);
> if (it.hasNext()) {
> continue; // try the next Proxy
> } else {
> throw ioex; // that was the last Proxy, time to fail
> }
> }
>
> Thanks,
> -Aleksey
>
