RFR: 8336843: Deprecate java.util.zip.ZipError for removal [v3]

Eirik Bjørsnøs eirbjo at openjdk.org
Fri Oct 4 15:38:39 UTC 2024


On Wed, 28 Aug 2024 08:11:08 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:

>> Please review this PR which suggests to deprecate the unused class `java.util.zip.ZipError` for removal.
>> 
>> The class has been unsed by OpenJDK since  the ZIP API was rewritten from native to Java in JDK 9.
>> 
>> I opted to not explain the reason for the deprecation in detail, but instead simply point to `ZipException` as an alternative. Should more explanation be desired, I could prepend that with a note saying that the class is unused since JDK 9.
>> 
>> A CSR for this API update has been drafted, I'll update the Specification section there once we reach a concensus on the deprecation note in this PR.
>> 
>> This deprecation was initially suggested here: https://mail.openjdk.org/pipermail/core-libs-dev/2024-June/125720.html
>
> Eirik Bjørsnøs 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 'master' into ziperror-deprecation
>  - Simplify the deprecation note by focusing on behavior in the current release
>  - Extend the deprecation note to mention that the error became obsolete in JDK 9 and to mention that code may be updated to catch the super class InternalError
>  - Update copyright year
>  - Deprecate java.util.zip.ZipError for removal

The CSR has been approved to mark `java.util.zip.ZipError` deprecated, for removal with spec changes identical to the current state of this PR.

I plan to integrate this early next week. Just in case there are still any lingering concerns. 

Thanks for all the patient reviews and comments, here and in the CSR!

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

PR Comment: https://git.openjdk.org/jdk/pull/20642#issuecomment-2393975006


More information about the core-libs-dev mailing list