On Tue, 1 Jun 2021 16:56:36 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:
>> Patrick Concannon has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8268056: Reverted changes to URLDecoder; reformatted change to FileTime > > src/java.base/share/classes/java/net/URLDecoder.java line 228: > >> 226: // A trailing, incomplete byte encoding such as >> 227: // "%x" will cause an exception to be thrown >> 228: > > This change is a bit difficult to review and there doesn't appear much gain > in changing from a regular switch to a switch expression in this case. Maybe > we should just keep the regular switch here? OK. I've reverted change, as suggested. See 2f179b5 ------------- PR: https://git.openjdk.java.net/jdk/pull/4285
