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

Jaikiran Pai jpai at openjdk.org
Mon Mar 2 05:20:18 UTC 2026


On Sun, 1 Mar 2026 20:31:22 GMT, Andrey Turbanov <aturbanov 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 181:
> 
>> 179:         }
>> 180:         if (toThrow != null) {
>> 181:             throw  toThrow;
> 
> Suggestion:
> 
>             throw toThrow;

Done.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/29935#discussion_r2870548894


More information about the core-libs-dev mailing list