On Tue, 26 May 2026 12:53:08 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 incrementally with one additional > commit since the last revision: > > minor adjustment to class level javadoc > For JDK 27 or later, replace the class description as per the current > proposal. We can drop the spec update for the read method. The issue of an > I/O exception when reading beyond a trailer is problematic too so that might > have to be removed from the current wording. I've updated the PR to remove the mention of `InputStream.available()` in the specification of read method of the `GZIPInputStream`. This PR (nor any existing text) specified `IOException` when reading beyond a trailer, so that didn't require any additional updates to this PR. ------------- PR Comment: https://git.openjdk.org/jdk/pull/30925#issuecomment-4571914805
