Integrated: 8159: openUncompressedStream supports compressed inputs on all streams

Carter Kozak duke at openjdk.org
Wed Jan 3 18:15:50 UTC 2024


On Mon, 11 Dec 2023 21:50:16 GMT, Carter Kozak <duke at openjdk.org> wrote:

> Previously this function supported compressed inputs only when the provided stream supported mark/reset. By shifting the BufferedInputStream wrapper prior to compression checks, we can support consistent behavior regardless of the stream implementation.
> The new tests in this commit fail without the accompanying change to `IOToolkit.java`.
> 
> I'd be grateful if anyone could help create a jmc issue for this, I lack perms to create a ticket as described in the [contributing docs](https://github.com/openjdk/jmc/blob/master/CONTRIBUTING.md#pull-requests).

This pull request has now been integrated.

Changeset: dba85091
Author:    Carter Kozak <ckozak at ckozak.net>
Committer: Marcus Hirt <hirt at openjdk.org>
URL:       https://git.openjdk.org/jmc/commit/dba850913b99f02660bca564fcae8b1cef33cbb1
Stats:     74 lines in 2 files changed: 52 ins; 6 del; 16 mod

8159: openUncompressedStream supports compressed inputs on all streams

Reviewed-by: bdutheil, hirt

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

PR: https://git.openjdk.org/jmc/pull/539


More information about the jmc-dev mailing list