RFR: 8189366: SocketInputStream.available() should check for eof

[vyom tewari]

Hi Chris,

Thanks for review, please find the updated 
webrev(http://cr.openjdk.java.net/~vtewari/8189366/webrev0.1/index.html) 
where i included the test.

Thanks,

Vyom


On Wednesday 10 October 2018 09:50 PM, Chris Hegarty wrote:
> Vyom,
>
> On 10/10/18 14:16, vyom tewari wrote:
>> Hi All,
>>
>> Please provide your review comments on below fix.
>>
>> Thanks,
>>
>> Vyom
>>
>>
>> On Friday 27 October 2017 11:31 AM, vyom tewari wrote:
>
> Wow, is it really almost a year since this was first
> discussed!
>
>>>  ...
>>>> http://cr.openjdk.java.net/~vtewari/8189366/webrev0.0/index.html 
>
> To satisfy myself, I put together a test, ran it through
> Oracle's build and test infra, both with and without your
> proposed changes, and it passes ok. This should give more
> confidence that there is no inadvertant behavior change
> resulting from your changes.
>
> http://cr.openjdk.java.net/~chegar/8189366.test/
>
> Can you please include this test as part of your push?
>
> Thanks,
> -Chris.
>
> P.S. The test amends an existing test with additional
> coverage, without interfering too much with the original
> test.