Integrated: 8377983: (zipfs) ZipFileSystem.initCEN needlessly reads END header

Eirik Bjørsnøs eirbjo at openjdk.org
Wed Feb 18 15:22:57 UTC 2026


On Mon, 16 Feb 2026 11:38:03 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:

> Please review this cleanup PR which updates `ZipFileSystem.initCEN` to not include the END header when reading the CEN bytes into memory.
> 
> The inclusion of the (unused) END header is a remnant from conversion from native to Java in Java 9 and  was already fixed for the corresponding `ZipFile.Source.initCEN` via JDK-8339711.
> 
> Fixing this also in `ZipFileSystem` cleans up the code locally but more importantly brings the implemenation more into alignment with the `ZipFile`.
> 
> Pure refactoring / cleanup, `noreg-cleanup`.

This pull request has now been integrated.

Changeset: d02ac57e
Author:    Eirik Bjørsnøs <eirbjo at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/d02ac57e8469ac77cc4f53de77107a278ac5f346
Stats:     8 lines in 1 file changed: 0 ins; 0 del; 8 mod

8377983: (zipfs) ZipFileSystem.initCEN needlessly reads END header

Reviewed-by: lancea

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

PR: https://git.openjdk.org/jdk/pull/29741


More information about the core-libs-dev mailing list