<i18n dev> 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 i18n-dev mailing list