On Sat, 24 Aug 2024 10:49:56 GMT, Eirik Bjørsnøs <eir...@openjdk.org> wrote:

> Please review this refactoring PR which moves the `ZipEntry.flag` field back 
> to `ZipOutputStream.XEntry`.
> 
> Moving this field will save four bytes from the `ZipEntry` object size and 
> also saves an unneccessary read in `ZipFile.getZipEntry`.
> 
> Testing:
> 
> This PR is a refactoring of existing code and does not update any tests. I 
> added the label `noreg-cleanup` to the JBS issue.
> 
> The following runs clean:
> 
> 
> make test TEST="test/jdk/java/util/zip"
> make test TEST="test/jdk/java/util/jar"
> 
> 
> Performance:
> 
> The JMH benchmark `java.util.zip.ZipFileGetEntry.getEntryHit` show a small 
> but consistent improvement (2-3%).

This pull request has been closed without being integrated.

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

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

Reply via email to