Re: RFR:8212114 Reconsider the affect on closed streams resulting from 8189366

2018-10-19 Thread Daniel Fuchs
Hi Vyom, This looks good to me as well. best regards, -- daniel On 18/10/2018 11:05, vyom tewari wrote: Hi All, Please review the below fix. Webrev: http://cr.openjdk.java.net/~vtewari/8212114/webrev0.0/index.html bugId: https://bugs.openjdk.java.net/browse/JDK-8212114 this change the rev

Re: RFR:8212114 Reconsider the affect on closed streams resulting from 8189366

2018-10-19 Thread Chris Hegarty
> On 18 Oct 2018, at 11:05, vyom tewari wrote: > > Hi All, > > Please review the below fix. > > Webrev: http://cr.openjdk.java.net/~vtewari/8212114/webrev0.0/index.html > Looks good to me. Thanks Vyom. -Chris.

RFR:8212114 Reconsider the affect on closed streams resulting from 8189366

2018-10-18 Thread vyom tewari
Hi All, Please review the below fix. Webrev: http://cr.openjdk.java.net/~vtewari/8212114/webrev0.0/index.html bugId: https://bugs.openjdk.java.net/browse/JDK-8212114 this change the revert the effect of JDK-8189366,  restore the existing(SocketInputStream:available() will throw the socketExce