On Mon, 14 Aug 2023 21:16:27 GMT, Volker Simonis <simo...@openjdk.org> 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

Reply via email to