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