> 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 23 additional commits since the 
last revision:

 - fix copyright year on BasicGZIPInputStreamTest.java
 - "or after" -> "or after them"
 - Alan's review - remove use of implSpec from GZIPInputStream.read()
 - merge latest from master branch
 - 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
 - ... and 13 more: https://git.openjdk.org/jdk/compare/62632e51...bb2af7c1

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

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

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

  Stats: 8945 lines in 281 files changed: 5450 ins; 2203 del; 1292 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