RFR: 8310837: Use ByteArrayLittleEndian in java.util.zip [v8]

Glavo duke at openjdk.org
Thu May 9 12:40:59 UTC 2024


On Wed, 28 Feb 2024 14:21:16 GMT, Glavo <duke at openjdk.org> wrote:

>> Using `ByteArrayLittleEndian` is simpler and faster.
>> 
>> `make test TEST="micro:java.util.zip.ZipFileOpen"`:
>> 
>> 
>>   Benchmark                     (size)  Mode  Cnt      Score      Error  Units
>> - ZipFileOpen.openCloseZipFile     512  avgt   15  39052.832 ±  107.496  ns/op
>> + ZipFileOpen.openCloseZipFile     512  avgt   15  36275.539 ±  663.193  ns/op
>> - ZipFileOpen.openCloseZipFile    1024  avgt   15  77106.494 ± 4159.300  ns/op
>> + ZipFileOpen.openCloseZipFile    1024  avgt   15  71955.013 ± 2296.050  ns/op
>
> Glavo has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains eight additional commits since the last revision:
> 
>  - Merge branch 'openjdk:master' into zip-utils
>  - Add tests
>  - Merge branch 'openjdk:master' into zip-utils
>  - Merge branch 'openjdk:master' into zip-utils
>  - Merge branch 'openjdk:master' into zip-utils
>  - Merge branch 'openjdk:master' into zip-utils
>  - Merge branch 'openjdk:master' into zip-utils
>  - use ByteArrayLittleEndian in ZipUtils

I've been busy working on other projects lately, so I've put these PRs on hold. If anyone cares to take a look at them I'll be right back.

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

PR Comment: https://git.openjdk.org/jdk/pull/14632#issuecomment-2102583431


More information about the core-libs-dev mailing list