RFR: 8317678: Fix up hashCode() for ZipFile.Source.Key [v9]
Jaikiran Pai
jpai at openjdk.org
Mon Oct 23 12:00:39 UTC 2023
On Mon, 23 Oct 2023 09:14:06 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:
>
> Update code comments
test/jdk/java/util/zip/ZipFile/ZipSourceCache.java line 80:
> 78: HashMap internalMap;
> 79: int numSources;
> 80: try (ZipFile zipFile = new ZipFile(ZIPFILE_NAME)) {
Do you think changing this to:
new ZipFile(relativeFile)
would be better and more clear about the intent? A few lines below, we use `new ZipFile(absoluteFile)` which I think is good because it makes it clear what is being done.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16115#discussion_r1368547393
More information about the core-libs-dev
mailing list