RFR: 8299748: java/util/zip/Deinflate.java failing on s390x [v5]

Amit Kumar amitkumar at openjdk.org
Thu Apr 27 07:24:24 UTC 2023


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

  comment suggestion from @jaikiran

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

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

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

  Stats: 14 lines in 1 file changed: 11 ins; 0 del; 3 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


More information about the core-libs-dev mailing list