On Fri, 28 Apr 2023 11:18:56 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:
> 
>   added space

Marked as reviewed by aturbanov (Committer).

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

PR Review: https://git.openjdk.org/jdk/pull/12283#pullrequestreview-1405758295

Reply via email to