Please review this small enhancement which makes `ZipFile.Source.Key` capture the relevant `BasicFileAttributes` fields in its constructor instead of calling into `BasicFileAttributes` for each `hashCode` or `equals` invocation.
This reduces the memory footprint of `Key` instances. It should be performance positive since we avoid creating `FileTime` instances once per comparison and avoid synchronization getting the file key. Capturing fields early also improves readability by making it more obvious what this Key class is comparing. Pure refactoring, no tests updated, `noreg-trivial`. ------------- Commit messages: - Update copyright year - Capture relevant BasicFileAttributes fields in the Key constructor Changes: https://git.openjdk.org/jdk/pull/29411/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=29411&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8376294 Stats: 15 lines in 1 file changed: 5 ins; 2 del; 8 mod Patch: https://git.openjdk.org/jdk/pull/29411.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/29411/head:pull/29411 PR: https://git.openjdk.org/jdk/pull/29411
