RFR: 8268056: Update java.net and java.nio to use switch expressions [v6]

Alan Bateman alanb at openjdk.java.net
Thu Jun 3 14:18:46 UTC 2021


On Thu, 3 Jun 2021 10:12:03 GMT, Patrick Concannon <pconcannon at openjdk.org> wrote:

>> Hi,
>> 
>> Could someone please review my code for updating the code in the `java.net` and `java.nio` packages to make use of the switch expressions?
>> 
>> Kind regards,
>> Patrick
>
> 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 eight additional commits since the last revision:
> 
>  - 8268056: renamed variable in Files
>  - Merge remote-tracking branch 'origin/master' into JDK-8268056
>  - Merge remote-tracking branch 'origin/master' into JDK-8268056
>  - 8268056: Added yield to switch expression in Files
>  - 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

Updated patch looks okay. I'm not 100% sure that it's a significant improvement over the original code but I think this is the nature of more complex switch statements.

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

Marked as reviewed by alanb (Reviewer).

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


More information about the nio-dev mailing list