On Mon, 6 Mar 2023 11:07:26 GMT, Jaikiran Pai <j...@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