> 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:

  changes request by @turbanoff

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/12283/files
  - new: https://git.openjdk.org/jdk/pull/12283/files/543d136a..bbeb8b27

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

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 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