On Fri, 27 Feb 2026 10:32:09 GMT, Alan Bateman <[email protected]> wrote:
>> Jaikiran Pai has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains four additional >> commits since the last revision: >> >> - Alan's review - adjust the doc for flush() and finish() methods >> - remove extra space >> - merge latest from master branch >> - 8369181 (v1): InflaterOutputStream: writing after finish() results in a >> NullPointerException (NPE) instead of an IOException > > src/java.base/share/classes/java/util/zip/InflaterOutputStream.java line 224: > >> 222: >> 223: /** >> 224: * Forces writing any pending buffered decompressed data to the >> output stream, > > What you would think about dropping the term "Force" from this method, and > flush. They can simply say that they write any pending buffered decompress > data to the underlying output stream. Done, I've updated the PR to remove "Force" from those methods. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/29935#discussion_r2870548403
