Re: RFR JDK-8161091 Incorrect Stream.FlowControl implementation allows to send DataFrame even when window size was exhausted

2016-07-11 Thread Sergey Kuksenko
I am awfully sorry, previous fix was incorrect. Please, review the right version: http://cr.openjdk.java.net/~skuksenko/jep110/8161091/webrev.01/ On 07/08/2016 02:40 PM, Sergey Kuksenko wrote: Hi, Could you please review the following fix for JDK-8161091?

RE: RFR (S) JDK-8160174: java.net.NetworkInterface - fixes and improvements for network interface listing

2016-07-11 Thread Christos Zoulas
On Jul 11, 1:36pm, christoph.lan...@sap.com ("Langer, Christoph") wrote: -- Subject: RE: RFR (S) JDK-8160174: java.net.NetworkInterface - fixes and im | Hi Chris (or anyone who is holding a stake in the NetworkInterface native i= | mplementation), | | I've spent some time on cleaning up

RE: RFR (S) JDK-8160174: java.net.NetworkInterface - fixes and improvements for network interface listing

2016-07-11 Thread Langer, Christoph
Hi Chris (or anyone who is holding a stake in the NetworkInterface native implementation), I've spent some time on cleaning up NetworkInterface.c and came up with a bigger change: http://cr.openjdk.java.net/~clanger/webrevs/8160174.2/ With this I attempted to consolidate the interface listing

Re: RFR 8022580: sun.net.ftp.impl.FtpClient.nameList(String path) handle "null" incorrectly

2016-07-11 Thread Daniel Fuchs
Hi Svetlana, Have you thought of using a CountDownLatch (or some other synchronization mechanism like e.g. Semaphore) to wait for the server to be ready instead of: 55 int port = 0; 56 while (port == 0) { 57 Thread.sleep(500); 58

Re: RFR 8022580: sun.net.ftp.impl.FtpClient.nameList(String path) handle "null" incorrectly

2016-07-11 Thread Svetlana Nikandrova
Chris, may be you can find a minute to review: JBS: https://bugs.openjdk.java.net/browse/JDK-8022580 Latest webrev: http://cr.openjdk.java.net/~snikandrova/8022580/webrev.03/ Thank you, Svetlana On 07.07.2016 21:06, Svetlana

Re: RFR: 8160786: No CCC for public class java.net.http.AsyncSSlDelegate

2016-07-11 Thread Chris Hegarty
On 11/07/16 13:42, Michael McMahon wrote: Simple change. An implementation class was made public by mistake http://cr.openjdk.java.net/~michaelm/8160786/webrev.1/ Reviewed. -Chris.

RFR: 8160786: No CCC for public class java.net.http.AsyncSSlDelegate

2016-07-11 Thread Michael McMahon
Simple change. An implementation class was made public by mistake http://cr.openjdk.java.net/~michaelm/8160786/webrev.1/ Thanks Michael