Re: RFR: 8216986 Remove unused code from SocksSocketImpl

2019-01-24 Thread Michael McMahon
On 24/01/2019, 14:38, Alan Bateman wrote: On 24/01/2019 13:09, Michael McMahon wrote: I've updated the webrev at http://cr.openjdk.java.net/~michaelm/8216986/webrev.2/ to add some tests and also I found the same dubious implementation of getLocalPort() in HttpConnectSocketImpl.java. The test

Re: RFR: 8216986 Remove unused code from SocksSocketImpl

2019-01-24 Thread Alan Bateman
On 24/01/2019 13:09, Michael McMahon wrote: I've updated the webrev at http://cr.openjdk.java.net/~michaelm/8216986/webrev.2/ to add some tests and also I found the same dubious implementation of getLocalPort() in HttpConnectSocketImpl.java. The test infrastructure needed some cleanup also. Th

Re: RFR: 8216986 Remove unused code from SocksSocketImpl

2019-01-24 Thread Michael McMahon
I've updated the webrev at http://cr.openjdk.java.net/~michaelm/8216986/webrev.2/ to add some tests and also I found the same dubious implementation of getLocalPort() in HttpConnectSocketImpl.java. The test infrastructure needed some cleanup also. Thanks, Michael. On 23/01/2019, 14:30, Micha

Re: RFR: 8216986 Remove unused code from SocksSocketImpl

2019-01-23 Thread Michael McMahon
On 23/01/2019, 12:37, Alan Bateman wrote: On 23/01/2019 11:44, Michael McMahon wrote: Hi Could I get the following webrev reviewed please? It is just to remove dead code from SocksSocketImpl. Most of the code was an (unused) attempt to implement SOCKS for ServerSockets. getLocalPort() was po

Re: RFR: 8216986 Remove unused code from SocksSocketImpl

2019-01-23 Thread Alan Bateman
On 23/01/2019 11:44, Michael McMahon wrote: Hi Could I get the following webrev reviewed please? It is just to remove dead code from SocksSocketImpl. Most of the code was an (unused) attempt to implement SOCKS for ServerSockets. getLocalPort() was potentially buggy and should not override the su