> Hot on the heals of #29288, this PR replaces `ArrayDeque` with `ArrayList` > for the Inflater cache implementation in `ZipFile.CleanableResource`. > > With this PR, we change the order of the cache from a LIFO queue to a FIFO > stack backed by ArrayList. The order seems unimportant, and has indeed been > FIFO in the past when using `java.util.Vector`. Intuitively, it should be > better to return the most recently used Inflater. > > No tests updated, strict refactoring, `noreg-cleanup`.
Eirik Bjørsnøs has updated the pull request incrementally with one additional commit since the last revision: Remove null check of final field istreams which is never null ------------- Changes: - all: https://git.openjdk.org/jdk/pull/29430/files - new: https://git.openjdk.org/jdk/pull/29430/files/a3f06cdc..388306a7 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=29430&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=29430&range=01-02 Stats: 13 lines in 1 file changed: 1 ins; 2 del; 10 mod Patch: https://git.openjdk.org/jdk/pull/29430.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/29430/head:pull/29430 PR: https://git.openjdk.org/jdk/pull/29430
