On Fri, 19 Apr 2024 18:49:33 GMT, robert engels <d...@openjdk.org> 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

Reply via email to