> Please review this PR to use modern APIs and language features to simplify > equals, hashCode, and compareTo for in java.nio and implementation code. > > Please note, test results are pending. > > Additional notes: > > * This PR saves a volatile read in java.nio.file.attribute.AclEntry.hashCode. > Not that it's too important, but worth noting because of rearrangements. > > * java.nio.charset.Charset#compareTo seems **inconsistent** with equals. If > so, I cannot see where that inconsistency is specified. > > * Is this a **bug** in sun.nio.ch.FileKey#hashCode? Tell me if not, I'll > revert it. > > * This PR simplifies the tail of java.nio.file.attribute.FileTime.compareTo. > Unless I'm missing something, that comment in source above the affected lines > **seems** not to prohibit such a simplification. > > * sun.nio.fs.UnixFileStore#hashCode does not include entry.name(). While it's > not wrong, I wonder if it was on purpose. > > * Despite its title, this PR also and opportunistically refactors > sun.nio.fs.UnixPath.endsWith.
Pavel Rappo has updated the pull request incrementally with one additional commit since the last revision: Re-address feedback ------------- Changes: - all: https://git.openjdk.org/jdk/pull/14886/files - new: https://git.openjdk.org/jdk/pull/14886/files/a39780f5..8c171ae5 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=14886&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=14886&range=01-02 Stats: 4 lines in 2 files changed: 0 ins; 2 del; 2 mod Patch: https://git.openjdk.org/jdk/pull/14886.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/14886/head:pull/14886 PR: https://git.openjdk.org/jdk/pull/14886
