RFR: 8301463: Code in DatagramSocket still refers to resolved JDK-8237352 [v2]

Michael McMahon michaelm at openjdk.org
Mon Feb 13 10:34:08 UTC 2023


> Hi,
> 
> This appeared on the JDK 20 EC list last week. Initially I thought the comment and the usage of the NO_DELEGATE sentinel could be removed, but that is not the case as a special non-null value is still required.
> 
> That leaves a trivial change of just removing the comment in the two places.
> 
> Thanks,
> Michael.

Michael McMahon 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 three additional commits since the last revision:

 - Merge branch 'master' into datagram-8301463
 - remove code change, leaving comment only deleted
 - impl

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/12526/files
  - new: https://git.openjdk.org/jdk/pull/12526/files/f126ef15..d0653248

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

  Stats: 43020 lines in 876 files changed: 16628 ins; 11892 del; 14500 mod
  Patch: https://git.openjdk.org/jdk/pull/12526.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/12526/head:pull/12526

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


More information about the net-dev mailing list