> 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/9dd569c0...40a57962

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/30925/files
  - new: https://git.openjdk.org/jdk/pull/30925/files/a22c1b83..40a57962

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=30925&range=11
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=30925&range=10-11

  Stats: 3817 lines in 1231 files changed: 1737 ins; 415 del; 1665 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

Reply via email to