RFR: 8260617: Merge ZipFile encoding check with the initial hash calculation
Claes Redestad
redestad at openjdk.java.net
Fri Jan 29 09:56:57 UTC 2021
- Merge checkEncoding into the byte[]-based normalizedHash. The latter is only used from ZipFile.initCEN right after the checkEncoding today, so verifying this is equivalent is straightforward.
- Factor out the logic to calculate hash, check encoding etc into the addEntry method to allow JITs to chew off larger chunks of the logic early on
Roughly 1.2x speedup on the JarFileMeta microbenchmarks, which include the time required to open the jar file and thus exercising the optimized code.
Testing: tier1-4
-------------
Commit messages:
- Refactor ZipFile.initCEN, merging checkEncoding and normalizedHash
Changes: https://git.openjdk.java.net/jdk/pull/2306/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2306&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8260617
Stats: 132 lines in 2 files changed: 31 ins; 68 del; 33 mod
Patch: https://git.openjdk.java.net/jdk/pull/2306.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/2306/head:pull/2306
PR: https://git.openjdk.java.net/jdk/pull/2306
More information about the core-libs-dev
mailing list