On Fri, 28 Apr 2023 09:23:59 GMT, Amit Kumar <amitku...@openjdk.org> wrote:
>> 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: > > suggestion from @simonis Changes requested by simonis (Reviewer). test/jdk/java/util/zip/DeInflate.java line 164: > 162: out2 = baos.toByteArray(); > 163: if (n != len || > 164: !Arrays.equals(in, 0, len, out2, 0, len)) || There's a `)` too much. you should at least run it a single time before pushing :) ------------- PR Review: https://git.openjdk.org/jdk/pull/12283#pullrequestreview-1405576535 PR Review Comment: https://git.openjdk.org/jdk/pull/12283#discussion_r1180166033
