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