On Tue, 12 Dec 2023 11:47:35 GMT, Lance Andersen <lan...@openjdk.org> wrote:

>> Eirik Bjorsnos has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Use uppercase LOC in ZipException messages
>
> src/jdk.zipfs/share/classes/jdk/nio/zipfs/ZipFileSystem.java line 2579:
> 
>> 2577:                 }
>> 2578:                 if (LOCSIG(buf) != LOCSIG) {
>> 2579:                     throw new ZipException("invalid loc header (bad 
>> signature)");
> 
> Please change loc -> LOC

Fixed, including the ZipException for the preceding read check.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/17059#discussion_r1423888116

Reply via email to