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

Amit Kumar amitkumar at openjdk.org
Thu Mar 30 03:06:51 UTC 2023


On Mon, 6 Mar 2023 11:07:26 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>Now coming to this proposed patch, now that you are using local ByteArrayOutputStream(s) for the deflate/inflate part in this check() method, I think the out1 and out2 should no longer be needed in this method and thus the method signature can be changed to remove these params. I see that this is the only place where this method is used, so changing the method signature of check() should be OK and shouldn't impact other parts of the test.

@jaikiran I've removed these parameters, not sure this is the correct way of doing this. But please let me know if you've different idea in mind. 

Thanks for your patience

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

PR Comment: https://git.openjdk.org/jdk/pull/12283#issuecomment-1489616583


More information about the core-libs-dev mailing list