<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">I am the maintainer of HMCL and we need to catch ZipError in our program because our program needs to be compatible with Java 8:<br></div><div dir="ltr"><br></div><div dir="ltr"><a href="https://github.com/HMCL-dev/HMCL/blob/85b68ad135267bc33e03c3624b1bced9b7804c39/HMCLCore/src/main/java/org/jackhuang/hmcl/util/io/CompressingUtils.java#L218-L220">https://github.com/HMCL-dev/HMCL/blob/85b68ad135267bc33e03c3624b1bced9b7804c39/HMCLCore/src/main/java/org/jackhuang/hmcl/util/io/CompressingUtils.java#L218-L220</a><br></div><div dir="ltr"><br></div><div dir="ltr">I'd love to see it deprecated, but I'm sure there are some use cases for it, so please evaluate the risks carefully.<br></div><div dir="ltr"><br></div><div>Glavo</div></div></div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Jun 30, 2024 at 5:42 PM Eirik Bjørsnøs <<a href="mailto:eirbjo@gmail.com">eirbjo@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi!<div><br></div><div>The java.util.zip.ZipError class seems unused in OpenJDK. I assume this is legacy from the native ZIP implementation in Java 8.</div><div><br></div><div>This exception class extends InternalError and seems to have been added in Java 6 to help compatibility with existing code catching InternalError (JDK-4615343)</div><div><br></div><div>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. </div><div><br></div><div>Given that this class has now fallen out of use in OpenJDK, should we:</div><div><br></div><div>1: Deprecate it</div><div>2: Deprecate it for removal</div><div>3: Do nothing, keeping it around has a low cost </div><div>4: Something else </div><div><br></div><div>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.</div><div><br></div><div>Thanks,</div><div>Eirik.</div></div>
</blockquote></div>