On Tue, 11 Aug 2026 11:52:18 GMT, Jaikiran Pai <[email protected]> wrote:
>> Volkan Yazici has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains five additional
>> commits since the last revision:
>>
>> - Merge remote-tracking branch 'upstream/master' into hsDrainAmount
>> - Convert `NoContentReturningHandler` to a class
>> - Fix comment on `SocketException("EOF")`
>> - Fix `@bug` tags
>> - Fix the `eof` handling in `FixedLengthInputStream`
>
> test/jdk/com/sun/net/httpserver/property/DrainAmountPropertyTest.java line
> 174:
>
>> 172: var inputStream = clientSocket.getInputStream();
>> 173: assertEquals("HTTP/1.1 204 No Content",
>> readUntilCrLf(inputStream));
>> 174: while (!readUntilCrLf(inputStream).isEmpty());
>
> I haven't grasped what this while loop is for. Can a brief comment be added
> for it?
Skip over field-lines until it reaches the empty lines that marks the end of
the field-line section.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/32234#discussion_r3758543532
