RFR: 8291956: Simplify the loop condition in sun.net.httpserver.Request constructor [v4]

Chris Hegarty chegar at openjdk.org
Fri Aug 5 19:02:11 UTC 2022


On Fri, 5 Aug 2022 15:07:17 GMT, thyecust <duke at openjdk.org> wrote:

>> The condition at line 57 (after while) will evaluate to false
>> if startLine == null, so the previous if-condition is covered.
>
> thyecust 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 six additional commits since the last revision:
> 
>  - Merge branch 'openjdk:master' into if
>  - Merge branch 'openjdk:master' into if
>  - update the Copyright year
>  - simplify loop contidition
>  - Merge branch 'openjdk:master' into if
>  - remove redundant if-condition after readLine()
>    
>    The condition at line 57 (after while) will evaluate to false
>    if startLine == null, so the previous if-condition is covered.

Marked as reviewed by chegar (Reviewer).

-------------

PR: https://git.openjdk.org/jdk/pull/9571


More information about the net-dev mailing list