> This change overrides mutator methods in the implementation returned by
> `Map.of().entrySet()` to throw `UnsupportedOperationException`.
Liam Miller-Cushon has updated the pull request incrementally with one
additional commit since the last revision:
Update test/jdk/java/util/Coll
> This change overrides mutator methods in the implementation returned by
> `Map.of().entrySet()` to throw `UnsupportedOperationException`.
Liam Miller-Cushon has updated the pull request incrementally with one
additional commit since the last revision:
Update copyright year
a
On Tue, 9 Jul 2024 22:43:05 GMT, Chen Liang wrote:
> All three files can have their copyright year updated.
Done
-
PR Comment: https://git.openjdk.org/jdk/pull/18522#issuecomment-2218861123
On Tue, 9 Jul 2024 22:23:57 GMT, Chen Liang wrote:
> Would you mind changing "Calling any mutator method on the Map will ..." to
> something like "... on the map or any derived view collection will ..." to
> emphasize our new consistency? (Our internal conversation agreed on this)
Done! Kevin
> This change overrides mutator methods in the implementation returned by
> `Map.of().entrySet()` to throw `UnsupportedOperationException`.
Liam Miller-Cushon has updated the pull request incrementally with one
additional commit since the last revision:
Update unmodifiable map j
> This change overrides mutator methods in the implementation returned by
> `Map.of().entrySet()` to throw `UnsupportedOperationException`.
Liam Miller-Cushon has updated the pull request with a new target base due to a
merge or a rebase. The incremental webrev excludes the unrelated c
On Tue, 9 Jul 2024 17:43:32 GMT, Chen Liang wrote:
>> `testCollMutatorsAlwaysThrow` expects a `Collection` (not e.g. a
>> `Collection>`). MOAT could be refactored to handle
>> that case. Do you think that's worth it, or have thoughts about what the
>> cleanest way to do that would be?
>
>
On Mon, 8 Jul 2024 20:39:38 GMT, Chen Liang wrote:
>> Liam Miller-Cushon 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 merge/rebase. The pull request contains
On Fri, 24 May 2024 15:32:09 GMT, Liam Miller-Cushon wrote:
> This change fixes a bug preventing javac from emitting
> 'compiler.warn.sun.proprietary' diagnostics if `--system` is set to a
> non-default value. The diagnostics are currently emitted for values of
>
On Tue, 4 Jun 2024 17:43:24 GMT, Liam Miller-Cushon wrote:
>> This change fixes a zip64 bug in the launcher that is prevent it from
>> reading the manifest of jars where the 'relative offset of local header'
>> field in the central directory entry is >4GB. As described in
ntral directory it is stored in a
> 'Zip64 Extended Information Extra Field'.
Liam Miller-Cushon 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 merge/rebase. The pull request contain
This change fixes a bug preventing javac from emitting
'compiler.warn.sun.proprietary' diagnostics if `--system` is set to a
non-default value. The diagnostics are currently emitted for values of
`--release`, and for the default value of `--system`.
The is a redo of
> This change overrides mutator methods in the implementation returned by
> `Map.of().entrySet()` to throw `UnsupportedOperationException`.
Liam Miller-Cushon has updated the pull request with a new target base due to a
merge or a rebase. The incremental webrev excludes the unrelated c
On Thu, 4 Apr 2024 16:57:40 GMT, Liam Miller-Cushon wrote:
>> This change fixes a zip64 bug in the launcher that is prevent it from
>> reading the manifest of jars where the 'relative offset of local header'
>> field in the central directory entry is >4GB. As described in
> This change overrides mutator methods in the implementation returned by
> `Map.of().entrySet()` to throw `UnsupportedOperationException`.
Liam Miller-Cushon has updated the pull request with a new target base due to a
merge or a rebase. The incremental webrev excludes the unrelated c
On Mon, 27 May 2024 12:18:23 GMT, Chen Liang wrote:
>> src/java.base/share/classes/java/util/ImmutableCollections.java line 1323:
>>
>>> 1321: @Override
>>> 1322: public int hashCode() {
>>> 1323: return MapN.this.hashCode();
>>
>> The hash
On Thu, 4 Apr 2024 16:57:40 GMT, Liam Miller-Cushon wrote:
>> This change fixes a zip64 bug in the launcher that is prevent it from
>> reading the manifest of jars where the 'relative offset of local header'
>> field in the central directory entry is >4GB. As described in
On Wed, 27 Mar 2024 17:36:28 GMT, Liam Miller-Cushon wrote:
> This change overrides mutator methods in the implementation returned by
> `Map.of().entrySet()` to throw `UnsupportedOperationException`.
Please keep this open
-
PR Comment: https://git.openjdk.org/jdk/pull
On Thu, 4 Apr 2024 09:42:48 GMT, Jaikiran Pai wrote:
>> Liam Miller-Cushon has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Add some more comments
>
> test/jdk/java/util/zip/Zip64Offsets.java line 1:
>
ntral directory it is stored in a
> 'Zip64 Extended Information Extra Field'.
Liam Miller-Cushon has updated the pull request incrementally with one
additional commit since the last revision:
Move test to test/jdk/tools/launcher
-
Changes:
- all: https://git.openjdk.org/jd
On Fri, 29 Mar 2024 18:25:18 GMT, Jiangli Zhou wrote:
>> it's in APPNOTE.TXT 4.5, the extra field structure is `header1+data1 +
>> header2+data2 . . .`, and we have to iterate through to see if there's an
>> entry that's a zip64 extended information extra field
>
> Thanks! That's helpful. I
ntral directory it is stored in a
> 'Zip64 Extended Information Extra Field'.
Liam Miller-Cushon has updated the pull request incrementally with one
additional commit since the last revision:
Add some more comments
-
Changes:
- all: https://git.openjdk.org/jdk/pull/18479/fi
On Fri, 29 Mar 2024 17:40:59 GMT, Jiangli Zhou wrote:
>> Liam Miller-Cushon has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Make cendsk an unsigned short
>
> src/java.base/share/native/libjli/parse_
On Fri, 29 Mar 2024 19:54:31 GMT, Jiangli Zhou wrote:
>> I think it doesn't matter, because the validation below only uses `cenoff`.
>> If `cenoff` fits in 32 bits, we don't need to read the zip64 extra info.
>
> Thanks for the explanation. Could you please add a comment with the info
> above
On Sat, 30 Mar 2024 17:28:11 GMT, Jiangli Zhou wrote:
>> I think that's similar idea to one of the alternatives I mentioned earlier,
>> won't that allocate for every central directory entry? This callsite has
>> already read the data we need into a buffer, if we end up doing something
>> like
On Sat, 30 Mar 2024 03:21:39 GMT, Jiangli Zhou wrote:
>> The other loop uses `readAt` to read in additional data and advance through
>> the extra fields, this loop already has access to a buffer that contains all
>> of the data for the extra fields and doesn't need to do that.
>>
>> I
On Fri, 29 Mar 2024 18:24:23 GMT, Jiangli Zhou wrote:
>> Liam Miller-Cushon has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Maximum Zip64 extra field length is 32
>
> src/java.base/share/native/li
On Fri, 29 Mar 2024 18:28:37 GMT, Jiangli Zhou wrote:
>> Liam Miller-Cushon has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Maximum Zip64 extra field length is 32
>
> src/java.base/share/native/li
On Fri, 29 Mar 2024 18:03:13 GMT, Jiangli Zhou wrote:
>> There can be multiple extra fields, and the zip64 extended information may
>> not be the first one
>
> Could you please point to the related spec for the other `extra field`?
it's in APPNOTE.TXT 4.5, the extra field structure is
On Fri, 29 Mar 2024 17:43:36 GMT, Jiangli Zhou wrote:
>> Liam Miller-Cushon has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Make cendsk an unsigned short
>
> src/java.base/share/native/libjli/parse_
On Fri, 29 Mar 2024 17:32:11 GMT, Jiangli Zhou wrote:
>> Thanks for the catch, I had missed that the disk start number is 4 bytes and
>> not 8. I pushed a commit. I also removed some unused references to the disk
>> number, which is only being used to validate the size of the zip64 extended
ntral directory it is stored in a
> 'Zip64 Extended Information Extra Field'.
Liam Miller-Cushon has updated the pull request incrementally with one
additional commit since the last revision:
Maximum Zip64 extra field length is 32
-
Changes:
- all: https://git.openjdk.org/jd
On Wed, 27 Mar 2024 21:49:10 GMT, Chen Liang wrote:
>> This change overrides mutator methods in the implementation returned by
>> `Map.of().entrySet()` to throw `UnsupportedOperationException`.
>
> src/java.base/share/classes/java/util/ImmutableCollections.java line 1321:
>
>> 1319:
This change overrides mutator methods in the implementation returned by
`Map.of().entrySet()` to throw `UnsupportedOperationException`.
-
Commit messages:
- Use AbstractImmutableSet
- Throw UOE for all Map.of().entrySet() mutator methods
- 8328821: Make the ImmutableCollections
ntral directory it is stored in a
> 'Zip64 Extended Information Extra Field'.
Liam Miller-Cushon has updated the pull request incrementally with one
additional commit since the last revision:
Make cendsk an unsigned short
-
Changes:
- all: https://git.openjdk.org/jdk/pull/1
On Wed, 27 Mar 2024 03:18:42 GMT, Jiangli Zhou wrote:
>> Liam Miller-Cushon has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Fix disk number size
>
> src/java.base/share/native/libjli/manifest_inf
ntral directory it is stored in a
> 'Zip64 Extended Information Extra Field'.
Liam Miller-Cushon has updated the pull request incrementally with one
additional commit since the last revision:
Fix disk number size
-
Changes:
- all: https://git.openjdk.org/jdk/pull/18479/fi
On Wed, 27 Mar 2024 03:19:47 GMT, Jiangli Zhou wrote:
>> This change fixes a zip64 bug in the launcher that is prevent it from
>> reading the manifest of jars where the 'relative offset of local header'
>> field in the central directory entry is >4GB. As described in APPNOTE.TXT
>> 4.5.3, the
This change fixes a zip64 bug in the launcher that is prevent it from reading
the manifest of jars where the 'relative offset of local header' field in the
central directory entry is >4GB. As described in APPNOTE.TXT 4.5.3, the offset
is too large to be stored in the central directory it is
On Wed, 30 Nov 2022 18:48:30 GMT, Liam Miller-Cushon wrote:
> This causes jar to not compress the `META-INF/` directory entry, for
> consistency with the handling of other directory entries and compliance with
> `APPNOTE.TXT`, and for compatibility with other zip implementations.
On Thu, 1 Dec 2022 11:44:11 GMT, Lance Andersen wrote:
>> Liam Miller-Cushon has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Improve test
>
> test/jdk/tools/jar/ManifestDirectoryCompression.java line 8
> This causes jar to not compress the `META-INF/` directory entry, for
> consistency with the handling of other directory entries and compliance with
> `APPNOTE.TXT`, and for compatibility with other zip implementations.
Liam Miller-Cushon has updated the pull request incrementally
On Thu, 1 Dec 2022 11:44:07 GMT, Lance Andersen wrote:
>> Unfortunately it's recursing on `cleanup` in the lambda, so it can't throw
>> checked exceptions without more refactoring. This is imitating the recursive
>> deletion approach in another jar test, I'm happy to swap this out if you'd
>>
> This causes jar to not compress the `META-INF/` directory entry, for
> consistency with the handling of other directory entries and compliance with
> `APPNOTE.TXT`, and for compatibility with other zip implementations.
Liam Miller-Cushon has updated the pull request incrementally
On Wed, 30 Nov 2022 22:15:14 GMT, Lance Andersen wrote:
>> Liam Miller-Cushon has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Use TestNG, and assert on the compression method
>
> test/jdk/tools/jar/Manifest
> This causes jar to not compress the `META-INF/` directory entry, for
> consistency with the handling of other directory entries and compliance with
> `APPNOTE.TXT`, and for compatibility with other zip implementations.
Liam Miller-Cushon has updated the pull request incrementally
On Wed, 30 Nov 2022 18:48:30 GMT, Liam Miller-Cushon wrote:
> This causes jar to not compress the `META-INF/` directory entry, for
> consistency with the handling of other directory entries and compliance with
> `APPNOTE.TXT`, and for compatibility with other zip implementations
This causes jar to not compress the `META-INF/` directory entry, for
consistency with the handling of other directory entries and compliance with
`APPNOTE.TXT`, and for compatibility with other zip implementations.
-
Commit messages:
- 8297875: jar should not compress the manifest
48 matches
Mail list logo
