On Mon, 2 Mar 2026 17:42:26 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: * Writes out any pending buffered decompressed data to the >> underlying output stream, > > The updated API docs looks good. For finish, I think you can drop "out from > this sentence so that it's "Writes any pending ...". I think this would fit > better with the other method descriptions. Done. Updated the CSR too. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/29935#discussion_r2873949421
