RFR: 8322256: Define and document GZIPInputStream concatenated stream semantics [v8]
Archie Cobbs
acobbs at openjdk.org
Mon Jul 29 13:55:37 UTC 2024
On Mon, 29 Jul 2024 13:06:24 GMT, Lance Andersen <lancea at openjdk.org> wrote:
> So where does that leave us:
>
> Keep the code as is and document the current behavior
> Continue to add additional test coverage for the current API
> We probably do not need a new constructor given it probably adds no new additional value the existing API
Just so I understand... are you saying that you are OK with this class being fundamentally unreliable _and_ providing no way to avoid that unreliability? (Just to restate a previous example of this: if an underlying `IOException` occurs at just the wrong time, an application that wrote `GZIP-STREAM-1`, `GZIP-STREAM-2`, `GZIP-STREAM-3` could read back `GZIP-STREAM-1`, `GZIP-STREAM-2` and never know that anything was wrong.)
If that's _not_ what you're saying, then I don't the understand "We probably do not need a new constructor" part.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/18385#issuecomment-2256010579
More information about the core-libs-dev
mailing list