RFR: 8297976: Remove sun.net.ProgressMonitor and related classes [v3]

Daniel Jeliński djelinski at openjdk.org
Mon Dec 5 14:12:05 UTC 2022


> Please review this patch that removes progress monitoring classes used by UrlConnection.
> Since Java 9 these classes are not used in the JDK, and are not exported from java.base. If anyone was still using them, reimplementing them in user code should be pretty straightforward.
> 
> This PR also fixes the issue where MeteredStream finalizer could resurrect an unusable connection, causing unexpected exceptions in other parts of the code.
> 
> No new regression test; since we are removing the finalizer, I don't believe we will see the issue again. I can add a test for that if you think it still makes sense.
> 
> I had to adjust `ProxyModuleMapping.java` test which used `sun.net.ProgressListener` as an example of a module-private interface; I replaced it with another public interface from the same package.
> 
> Existing tier 1-3 tests continue to pass.

Daniel Jeliński has updated the pull request incrementally with one additional commit since the last revision:

  Remove double space
  
  Co-authored-by: Andrey Turbanov <turbanoff at gmail.com>

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/11474/files
  - new: https://git.openjdk.org/jdk/pull/11474/files/e0e157c7..8d7c6e5c

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

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/11474.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/11474/head:pull/11474

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


More information about the core-libs-dev mailing list