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

2023-12-14 Thread Jaikiran Pai
On Mon, 23 Oct 2023 16:12:45 GMT, Sean Coffey 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:

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

2023-12-13 Thread Eirik Bjorsnos
On Mon, 23 Oct 2023 16:12:45 GMT, Sean Coffey 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:

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

2023-10-23 Thread Sean Coffey
On Mon, 23 Oct 2023 16:12:45 GMT, Sean Coffey 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:

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

2023-10-23 Thread Jaikiran Pai
On Mon, 23 Oct 2023 16:12:45 GMT, Sean Coffey 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:

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

2023-10-23 Thread Sean Coffey
> 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 jmh test comments - Changes: - all:

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

2023-10-23 Thread Lance Andersen
On Mon, 23 Oct 2023 13:04:20 GMT, Sean Coffey 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:

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

2023-10-23 Thread Jaikiran Pai
On Mon, 23 Oct 2023 13:04:20 GMT, Sean Coffey 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:

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

2023-10-23 Thread Jaikiran Pai
On Mon, 23 Oct 2023 13:04:20 GMT, Sean Coffey 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:

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

2023-10-23 Thread Sean Coffey
On Mon, 23 Oct 2023 13:04:20 GMT, Sean Coffey 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:

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

2023-10-23 Thread Sean Coffey
On Mon, 23 Oct 2023 13:04:20 GMT, Sean Coffey 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:

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

2023-10-23 Thread Sean Coffey
> 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: Incorporate test review comments from Jai -

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

2023-10-23 Thread Jaikiran Pai
On Mon, 23 Oct 2023 09:14:06 GMT, Sean Coffey 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:

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

2023-10-23 Thread Jaikiran Pai
On Mon, 23 Oct 2023 09:14:06 GMT, Sean Coffey 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:

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

2023-10-23 Thread Jaikiran Pai
On Mon, 23 Oct 2023 09:14:06 GMT, Sean Coffey 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:

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

2023-10-23 Thread Jaikiran Pai
On Mon, 23 Oct 2023 10:39:11 GMT, Sean Coffey wrote: > ZipFile behaviour in the scenario where a file is re-opened and edited > while references to the old ZipFile exist. A "invalid LOC header" needs to be > thrown in cases where an old zip file mapping is still in use. This PR doesn't do any

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

2023-10-23 Thread Sean Coffey
On Mon, 23 Oct 2023 09:14:06 GMT, Sean Coffey 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:

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

2023-10-23 Thread Alan Bateman
On Mon, 23 Oct 2023 09:14:06 GMT, Sean Coffey 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:

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

2023-10-23 Thread Jaikiran Pai
On Mon, 23 Oct 2023 09:14:06 GMT, Sean Coffey 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:

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

2023-10-23 Thread Sean Coffey
> 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 - Changes: - all:

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

2023-10-20 Thread Alan Bateman
On Wed, 18 Oct 2023 15:59:13 GMT, Sean Coffey 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:

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

2023-10-18 Thread Lance Andersen
On Wed, 18 Oct 2023 15:59:13 GMT, Sean Coffey 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:

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

2023-10-18 Thread Sean Coffey
> 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 lastModifiedTime comment - Changes:

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

2023-10-18 Thread Alan Bateman
On Wed, 18 Oct 2023 10:09:29 GMT, Sean Coffey 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:

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

2023-10-18 Thread Lance Andersen
On Wed, 18 Oct 2023 10:09:29 GMT, Sean Coffey 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:

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

2023-10-18 Thread Sean Coffey
> 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: Testcase feedback from Lance - Changes: -

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

2023-10-17 Thread Lance Andersen
On Tue, 17 Oct 2023 19:55:19 GMT, Sean Coffey 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:

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

2023-10-17 Thread Sean Coffey
On Tue, 17 Oct 2023 19:55:19 GMT, Sean Coffey 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:

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

2023-10-17 Thread Sean Coffey
> 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

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

2023-10-17 Thread Sean Coffey
On Tue, 17 Oct 2023 08:50:15 GMT, Sean Coffey 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 with a new target base due to a > merge or a rebase. The

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

2023-10-17 Thread Sean Coffey
> 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 with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the

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

2023-10-16 Thread Alan Bateman
On Fri, 13 Oct 2023 12:18:41 GMT, Sean Coffey 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:

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

2023-10-16 Thread Alan Bateman
On Fri, 13 Oct 2023 12:18:41 GMT, Sean Coffey 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:

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

2023-10-16 Thread Jaikiran Pai
On Fri, 13 Oct 2023 12:18:41 GMT, Sean Coffey 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:

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

2023-10-16 Thread Jaikiran Pai
On Fri, 13 Oct 2023 12:18:41 GMT, Sean Coffey 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:

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

2023-10-13 Thread Lance Andersen
On Fri, 13 Oct 2023 12:18:41 GMT, Sean Coffey 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:

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

2023-10-13 Thread Sean Coffey
> 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 -

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

2023-10-13 Thread Sean Coffey
On Fri, 13 Oct 2023 08:45:07 GMT, Sean Coffey 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:

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

2023-10-13 Thread Sean Coffey
> 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: insert back lastmodified check. enhance testcase. more

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

2023-10-10 Thread Alan Bateman
On Tue, 10 Oct 2023 12:46:22 GMT, Sean Coffey 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:

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

2023-10-10 Thread Sean Coffey
On Tue, 10 Oct 2023 12:46:22 GMT, Sean Coffey 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:

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

2023-10-10 Thread Sean Coffey
> 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: incorporate review comments - Changes: -

Re: RFR: 8317678: Fix up hashCode() for ZipFile.Source.Key

2023-10-10 Thread Chen Liang
On Tue, 10 Oct 2023 08:32:28 GMT, Sean Coffey 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 1418: > 1416: > 1417:

Re: RFR: 8317678: Fix up hashCode() for ZipFile.Source.Key

2023-10-10 Thread Alan Bateman
On Tue, 10 Oct 2023 08:32:28 GMT, Sean Coffey wrote: > Fix up java.util.zip.ZipFile$Source hashCode() impl so that duplicate Source > objects aren't created for the same zip file. This is really ZipFile.Source.Key hashCode/equals being inconsistent so I think we should rename the JBS/PR title

Re: RFR: 8317678: Fix up hashCode() for ZipFile.Source.Key

2023-10-10 Thread Pavel Rappo
On Tue, 10 Oct 2023 08:32:28 GMT, Sean Coffey 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:

Re: RFR: 8317678: Fix up hashCode() for ZipFile.Source.Key

2023-10-10 Thread Alan Bateman
On Tue, 10 Oct 2023 08:32:28 GMT, Sean Coffey 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 1424: > 1422: return

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

2023-10-10 Thread Sean Coffey
Fix up java.util.zip.ZipFile$Source hashCode() impl so that duplicate Source objects aren't created for the same zip file. - Commit messages: - Initial changes Changes: https://git.openjdk.org/jdk/pull/16115/files Webrev: https://webrevs.openjdk.org/?repo=jdk=16115=00 Issue: