java.util.zip.ZipError seems unused

Lance Andersen lance.andersen at oracle.com
Sun Jun 30 12:54:06 UTC 2024


Hi Eirik,

The removal of ZipError from ZipFile/ZipFileSystem/ZipFileSystemProvider  occurred via 8145260 and 8037394 for JDK 9

The test should also be re-written at this point

Jai or I can make a pass to see if there are any external usages via a corpus search but I tend to doubt it

On Jun 30, 2024, at 3:20 AM, Eirik Bjørsnøs <eirbjo at gmail.com> wrote:

Hi!

The java.util.zip.ZipError class seems unused in OpenJDK. I assume this is legacy from the native ZIP implementation in Java 8.

This exception class extends InternalError and seems to have been added in Java 6 to help compatibility with existing code catching InternalError (JDK-4615343)

This change also introduced the TestZipError test, which verified that ZipError was thrown while enumerating a ZIP file which was changed after being opened. The reimplementation of the ZIP implementation to Java (JDK-8145260) updated this test to expect a ZipException instead of the ZipError.

Given that this class has now fallen out of use in OpenJDK, should we:

1: Deprecate it
2: Deprecate it for removal
3: Do nothing, keeping it around has a low cost
4: Something else

It would also be useful if someone with access to a large code corpus could search for usages of this class so we could assess compatibility concerns of removing it.

Thanks,
Eirik.

[oracle_sig_logo.gif]






Lance Andersen | Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/core-libs-dev/attachments/20240630/15b993d0/attachment-0001.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: oracle_sig_logo.gif
Type: image/gif
Size: 658 bytes
Desc: oracle_sig_logo.gif
URL: <https://mail.openjdk.org/pipermail/core-libs-dev/attachments/20240630/15b993d0/oracle_sig_logo-0001.gif>


More information about the core-libs-dev mailing list