On Mon, 14 Aug 2023 21:16:27 GMT, Volker Simonis <[email protected]> wrote:
>> Lance Andersen has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Added an additional test with with 0 extra header followed by an extra
>> header with a data size
>
> test/jdk/java/util/zip/ZipFile/ReadNonStandardExtraHeadersTest.java line 871:
>
>> 869: */
>> 870: private static Stream<Path> zipFilesToTest() {
>> 871: return Stream.of(VALID_APK, VALID_APACHE_COMPRESS_JAR,
>> VALID_ANT_JAR);
>
> And here you probably want to add `VALID_ANT_ZIP` in addition to
> `VALID_ANT_JAR`.
Yep, already caught that typo, forgot to save before I committed :-)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15273#discussion_r1293984884
