> 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 16 additional commits since the last revision: - 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 - specify that GZIPInputStream is not thread-safe - refer to section 2.2 of the RFC - remove additional sentence from constructor documentation - ... and 6 more: https://git.openjdk.org/jdk/compare/752b2d2d...a3c3f4d2 ------------- Changes: - all: https://git.openjdk.org/jdk/pull/30925/files - new: https://git.openjdk.org/jdk/pull/30925/files/3d7cd005..a3c3f4d2 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=30925&range=09 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=30925&range=08-09 Stats: 11134 lines in 259 files changed: 7877 ins; 1445 del; 1812 mod Patch: https://git.openjdk.org/jdk/pull/30925.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/30925/head:pull/30925 PR: https://git.openjdk.org/jdk/pull/30925
