On Fri, 19 Apr 2024 18:49:33 GMT, robert engels <[email protected]> wrote:
>> src/jdk.httpserver/share/classes/sun/net/httpserver/ChunkedOutputStream.java
>> line 142:
>>
>>> 140: try {
>>> 141: /* write any pending chunk data */
>>> 142: writeChunk();
>>
>> Isn't there a risk that you might write two empty chunks here if there are
>> no pending chunks?
>
> yes, let me fixed that.
Shouldn't you look at whether count > 0 as is done in flush() ?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18667#discussion_r1572782506
