On Mon, 14 Nov 2022 21:00:16 GMT, Brian Burkhalter <b...@openjdk.org> wrote:
> Nitpick: should the `InputStream` import precede that of `IOException`? Gonna leave this as is as IntelliJ keeps changing it and not worth fighting > src/java.base/share/classes/java/util/zip/ZipInputStream.java line 178: > >> 176: * Returns 0 when end of stream is detected for the current ZIP >> entry or >> 177: * {@link #closeEntry()} has been called on the current ZIP entry, >> otherwise >> 178: * return 1. > > Another nitpick: should this be `returns 1` to match `Returns 0` at line 176? > Or vice versa? Sure updated ------------- PR: https://git.openjdk.org/jdk/pull/10995