RFR: 8267587: Update java.util to use enhanced switch [v7]
Tagir F.Valeev
tvaleev at openjdk.java.net
Mon May 31 03:44:31 UTC 2021
> 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
More information about the security-dev
mailing list