RFR: 8317678: Fix up hashCode() for ZipFile.Source.Key [v4]

Lance Andersen lancea at openjdk.org
Fri Oct 13 16:16:58 UTC 2023


On Fri, 13 Oct 2023 12:18:41 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:
> 
>   minor test edits and comment updates

Thank you for driving this issue Sean.

Overall I think it looks good.

Please see my minor comment below which you can handle prior to pushing the PR

test/jdk/java/util/zip/ZipFile/ZipSourceCache.java line 63:

> 61: 
> 62:     @Test
> 63:     public static void test() throws Exception {

Perhaps give the test a better name and add a brief comment block at least to the test method as we have been trying to add more clarity to the new tests that we add

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

Marked as reviewed by lancea (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/16115#pullrequestreview-1667211269
PR Review Comment: https://git.openjdk.org/jdk/pull/16115#discussion_r1352165848


More information about the core-libs-dev mailing list