> DeInflate.java test fails on s390x platform because size for out1 array which 
> is responsible for storing the compressed data is insufficient. And being 
> unable to write whole compressed data on array, on s390 whole data can't be 
> recovered after compression. This PR updates the check method in the 
> DeInflate test to no longer rely on pre-defined lengths/sizes to determine 
> whether deflate followed by an inflate of data worked correctly. These sizes 
> can vary depending on the underlying zlib implementations. The updated check 
> method now uses a `ByteArrayOutputStream` to deflate into and then inflate 
> from. 
> 
> Thanks to @jaikiran for amazing PR description.

Amit Kumar has updated the pull request incrementally with one additional 
commit since the last revision:

  comments from Lance

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/12283/files
  - new: https://git.openjdk.org/jdk/pull/12283/files/bce89892..625ad4bb

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=12283&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=12283&range=02-03

  Stats: 44 lines in 1 file changed: 18 ins; 13 del; 13 mod
  Patch: https://git.openjdk.org/jdk/pull/12283.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/12283/head:pull/12283

PR: https://git.openjdk.org/jdk/pull/12283

Reply via email to