RFR: 8260617: Merge ZipFile encoding check with the initial hash calculation [v4]

Claes Redestad redestad at openjdk.java.net
Tue Feb 2 10:40:01 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

Claes Redestad has updated the pull request incrementally with one additional commit since the last revision:

  Refactor out specific ZipException from checkedHash

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/2306/files
  - new: https://git.openjdk.java.net/jdk/pull/2306/files/bb1659e3..695273fb

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2306&range=03
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2306&range=02-03

  Stats: 75 lines in 2 files changed: 18 ins; 14 del; 43 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