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

thyecust duke at openjdk.org
Fri Aug 5 13:36:47 UTC 2022


> 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 five additional commits since the last revision:

 - 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.

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/9571/files
  - new: https://git.openjdk.org/jdk/pull/9571/files/d28bd1c0..368a6666

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=9571&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=9571&range=01-02

  Stats: 17457 lines in 745 files changed: 9703 ins; 5748 del; 2006 mod
  Patch: https://git.openjdk.org/jdk/pull/9571.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/9571/head:pull/9571

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


More information about the net-dev mailing list