RFR: 8306040: HttpResponseInputStream.available() returns 1 on empty stream
Vyom Tewari
vtewari at openjdk.org
Tue Jul 11 11:13:05 UTC 2023
On Mon, 10 Jul 2023 07:42:35 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> Please review the code change for [JDK-8306040](https://bugs.openjdk.org/browse/JDK-8306040). In the overridden "available" method of "HttpResponseInputStream" we are returning 1 after exploring all the code path.
>
> src/java.net.http/share/classes/jdk/internal/net/http/ResponseSubscribers.java line 541:
>
>> 539: Iterator<?> iterator = currentListItr;
>> 540: if (iterator != null && iterator.hasNext()) return 1;
>> 541: if (!buffers.isEmpty() && buffers.peek() != LAST_LIST ) return 1;
>
> Should probably be:
>
>
> if (iterator != null && iterator.hasNext() && Utils.hasRemaining(buffers)) return 1;
do you mean
if (iterator != null && iterator.hasNext() && Utils.hasRemaining(buffers.peek())) return 1; ?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14810#discussion_r1259581505
More information about the net-dev
mailing list