RFR JDK-8041563: ProblemList.txt updates

2014-04-23 Thread hefeng cui
Hi, Please review the my code changes for JDK-8041563 Here is the webrev link : http://cr.openjdk.java.net/~ewang/michael/JDK-8041563/webrev.00/ Thanks, Michael Cui

Re: RFR JDK-8041563: ProblemList.txt updates

2014-04-23 Thread Chris Hegarty
Looks ok to me Michael. I can sponsor this change for you. -Chris. On 23/04/14 10:01, hefeng cui wrote: Hi, Please review the my code changes for JDK-8041563 Here is the webrev link : http://cr.openjdk.java.net/~ewang/michael/JDK-8041563/webr

Re: RFR JDK-8041563: ProblemList.txt updates

2014-04-23 Thread hefeng cui
Hi Chris, Thanks for your review and sponsor! Regards, Michael Cui On 2014/4/23 17:26, Chris Hegarty wrote: Looks ok to me Michael. I can sponsor this change for you. -Chris. On 23/04/14 10:01, hefeng cui wrote: Hi, Please review the my code changes for JDK-8041563

Re: Proxied https connection reuse by sun.net.www.http.HttpClient can send CONNECT to the destination server

2014-04-23 Thread Chris Hegarty
..and finally pushed to JDK 9. http://hg.openjdk.java.net/jdk9/dev/jdk/rev/ea0d59f39e8f I will request backports to both 8u and 7u. -Chris. On 15/04/14 18:54, Steven Lawrance wrote: Thanks! Hopefully, it can also get backported to JDK7, but it's good that it can get fixed in at least JDK9.

RFR: 8041397: Lint regression in java.net.SocketOption

2014-04-23 Thread Michael McMahon
Could I get the following small change reviewed please? It fixes a number of recently introduced lint warnings http://cr.openjdk.java.net/~michaelm/8041397/webrev.1/ Thanks Michael.

Re: RFR: 8041397: Lint regression in java.net.SocketOption

2014-04-23 Thread roger riggs
Hi Michael, Looks fine. Roger On 4/23/2014 11:56 AM, Michael McMahon wrote: Could I get the following small change reviewed please? It fixes a number of recently introduced lint warnings http://cr.openjdk.java.net/~michaelm/8041397/webrev.1/ Thanks Michael.

Re: RFR: 8041397: Lint regression in java.net.SocketOption

2014-04-23 Thread Chris Hegarty
> On 23 Apr 2014, at 17:07, roger riggs wrote: > > Hi Michael, > > Looks fine. +1 -Chris. > > Roger > > >> On 4/23/2014 11:56 AM, Michael McMahon wrote: >> Could I get the following small change reviewed please? >> >> It fixes a number of recently introduced lint warnings >> >> http://c