<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body style="overflow-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;">
Hi Eirik,
<div><br>
</div>
<div>The removal of ZipError from ZipFile/ZipFileSystem/ZipFileSystemProvider occurred via <span style="white-space: pre; background-color: rgb(255, 255, 255);">8145260 and
</span><span style="white-space: pre; background-color: rgb(255, 255, 255);">8037394 for JDK 9</span></div>
<div><span style="white-space: pre; background-color: rgb(255, 255, 255);"><br>
</span></div>
<div><span style="white-space: pre; background-color: rgb(255, 255, 255);">The test should also be re-written at this point</span></div>
<div><span style="white-space: pre; background-color: rgb(255, 255, 255);"><br>
</span></div>
<div><span style="white-space: pre; background-color: rgb(255, 255, 255);">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<br id="lineBreakAtBeginningOfMessage">
</span>
<div><br>
<blockquote type="cite">
<div>On Jun 30, 2024, at 3:20 AM, Eirik Bjørsnøs <eirbjo@gmail.com> wrote:</div>
<br class="Apple-interchange-newline">
<div>
<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>
</div>
</blockquote>
</div>
<br>
<div><span><img alt="oracle_sig_logo.gif" src="cid:E1C4E2F0-ECD0-4C9D-ADB4-B16CA7BCB7FC@home"></span>
<div dir="auto" style="font-family: Helvetica; font-size: 18px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none; caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;">
<div><br class="Apple-interchange-newline">
<br class="Apple-interchange-newline">
<br class="Apple-interchange-newline">
<br>
<br>
<br>
Lance Andersen | Principal Member of Technical Staff | +1.781.442.2037</div>
<div>Oracle Java Engineering <br>
1 Network Drive <br>
Burlington, MA 01803<br>
Lance.Andersen@oracle.com</div>
</div>
<br class="Apple-interchange-newline" style="font-family: Helvetica; font-size: 18px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none; caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0);">
<br class="Apple-interchange-newline" style="font-family: Helvetica; font-size: 18px; font-style: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none; caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0);">
</div>
<br>
</div>
</body>
</html>