On Tue, 1 Jun 2021 16:58:12 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:
>> Patrick Concannon 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 four additional >> commits since the last revision: >> >> - 8268056: reverted changes to FileTime >> - Merge remote-tracking branch 'origin/master' into JDK-8268056 >> - 8268056: Reverted changes to URLDecoder; reformatted change to FileTime >> - 8268056: Update java.net and java.nio to use switch expressions > > src/java.base/share/classes/java/nio/file/attribute/FileTime.java line 240: > >> 238: Long.MAX_VALUE / SECONDS_PER_HOUR); >> 239: case MINUTES -> secs = scale(value, SECONDS_PER_MINUTE, >> 240: Long.MAX_VALUE / SECONDS_PER_MINUTE); > > It would be nicer to keep the second line aligned with the opening > parenthesis, as it was before. Change made in commit 2f179b5, but reverted all changes made to file as per Alan's request. See 23f53c5 ------------- PR: https://git.openjdk.java.net/jdk/pull/4285
