On Mon, 24 May 2021 04:20:23 GMT, Tagir F. Valeev <tval...@openjdk.org> wrote:

> 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?

src/java.base/share/classes/java/util/jar/JarFile.java line 201:

> 199:                 MULTI_RELEASE_FORCED = true;
> 200:             }
> 201:             default -> {

Here, explicit `case "true"` disappeared. Not sure if it's important to keep it.

-------------

PR: https://git.openjdk.java.net/jdk/pull/4161

Reply via email to