RFR: 8317678: Fix up hashCode() for ZipFile.Source.Key [v6]
Lance Andersen
lancea at openjdk.org
Tue Oct 17 20:13:48 UTC 2023
On Tue, 17 Oct 2023 19:55:19 GMT, Sean Coffey <coffeys at openjdk.org> wrote:
>> Fix up java.util.zip.ZipFile$Source hashCode() impl so that duplicate Source objects aren't created for the same zip file.
>
> Sean Coffey has updated the pull request incrementally with one additional commit since the last revision:
>
> remove getCanonicalFile calls, update ZipFile jmh benchmark test, tidy up junit test
Latest updates look good.
test/jdk/java/util/zip/ZipFile/ZipSourceCache.java line 103:
> 101: // the old Source in use for old file, should no longer map correctly
> 102: IOException ioe = assertThrows(IOException.class, () -> readZipFileContents(absoluteZipFile));
> 103: assertEquals("ZipFile invalid LOC header (bad signature)", ioe.getMessage());
Perhaps put the expected message in a constant in the event we need to change it at some point it is easier to find
-------------
Marked as reviewed by lancea (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/16115#pullrequestreview-1683270909
PR Review Comment: https://git.openjdk.org/jdk/pull/16115#discussion_r1362698792
More information about the core-libs-dev
mailing list