On Tue, 10 Oct 2023 08:32:28 GMT, Sean Coffey <coff...@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. src/java.base/share/classes/java/util/zip/ZipFile.java line 1444: > 1442: } else { > 1443: return ((int)(t ^ (t >>> 32))) + file.hashCode(); > 1444: } Consider this trivial simplification: Suggestion: Object fk = attrs.fileKey(); return Long.hashCode(t) + (fk != null ? fk.hashCode() : file.hashCode()); ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16115#discussion_r1351918933