RFR: 8323645: Remove unused internal sun.net.www.protocol.jar.URLJarFileCallBack interface [v2]

Jaikiran Pai jpai at openjdk.org
Thu Jan 25 09:39:39 UTC 2024


On Sat, 13 Jan 2024 08:46:50 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Can I please get a review of this change which removes unused code from `sun.net.www.protocol.jar.URLJarFile` and removes `sun.net.www.protocol.jar.URLJarFileCallBack`?
>> 
>> As noted in the linked issue https://bugs.openjdk.org/browse/JDK-8323645 the `sun.net.www.protocol.jar.URLJarFileCallBack` was relevant and used only by Java Plugin code (the "deploy" module in JDK 8) which hasn't been in the JDK for several releases now. Effectively this interface and its sole usage in an internal class of the JDK has been a leftover.
>> 
>> The commit in this PR removes that leftover code. No new tests have been added, given the nature of this change. tier1, tier2, tier3 continue to pass with this change.
>
> Jaikiran Pai 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 two additional commits since the last revision:
> 
>  - merge latest from master branch
>  - 8323645: Remove unused internal sun.net.www.protocol.jar.URLJarFileCallBack interface

Thank you Alan and Daniel for the reviews.

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

PR Comment: https://git.openjdk.org/jdk/pull/17396#issuecomment-1909753226


More information about the net-dev mailing list