> Inspired by PR#4088. Most of the changes are done automatically using > IntelliJ IDEA refactoring. Some manual adjustments are also performed, > including indentations, moving comments, extracting common cast out of switch > expression branches, etc. > > I also noticed that there are some switches having one branch only in > JapaneseImperialCalendar.java. Probably it would be better to replace them > with `if` statement?
Tagir F. Valeev has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains ten commits: - Rollback changes in j.u.concurrent (including formatting) - Rollback changes in j.u.concurrent - JapaneseImperialCalendar: use switch expressions - Use yield in java.util.Calendar.Builder.build - More vertical alignment - Vertical alignment for single-line switch rules - Indent some lines to make GitHub diff happier - Unindent switch cases to simplify the review - 8267587: Update java.util to use enhanced switch ------------- Changes: https://git.openjdk.java.net/jdk/pull/4161/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=4161&range=06 Stats: 884 lines in 15 files changed: 108 ins; 328 del; 448 mod Patch: https://git.openjdk.java.net/jdk/pull/4161.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/4161/head:pull/4161 PR: https://git.openjdk.java.net/jdk/pull/4161