RFR: 8369181: InflaterOutputStream: writing after finish() results in IllegalStateException instead of an IOException [v2]

[Jaikiran Pai]

On Fri, 27 Feb 2026 10:32:09 GMT, Alan Bateman <alanb at openjdk.org> 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