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