On Sat, 16 May 2026 07:21:09 GMT, Jaikiran Pai <[email protected]> wrote:
>> Can I please get a review of this doc and test-only change for >> https://bugs.openjdk.org/browse/JDK-8322256? >> >> The `java.util.zip.GZIPInputStream` has been in the JDK since Java 1.1. >> However, its specification hasn't been clear on how it behaves, especially >> when a `InputStream` consists of more than one GZIP member. >> >> The commit in this PR updates the specification of this class to match its >> current (long standing) implementation. >> >> A new jtreg test has been introduced to verify this behaviour. >> >> I'll draft a CSR shortly. >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > 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 19 additional > commits since the last revision: > > - Alan's review - update the class level doc > - merge latest from master branch > - merge latest from master branch > - merge latest from master branch > - minor adjustments > - Lance's review > - merge latest from master branch > - introduce test to verify read() throws IOException when invoked on closed > stream > - specify existing behaviour of throwing IOException when stream is already > closed > - merge latest from master branch > - ... and 9 more: https://git.openjdk.org/jdk/compare/aa58c9e3...40a57962 test/jdk/java/util/zip/GZIP/BasicGZIPInputStreamTest.java line 2: > 1: /* > 2: * Copyright (c) 2024, Oracle and/or its affiliates. All rights reserved. Update this :) ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30925#discussion_r3297698149
