RFR: 8268056: Update java.net and java.nio to use switch expressions [v2]
Alan Bateman
alanb at openjdk.java.net
Tue Jun 1 17:49:20 UTC 2021
On Tue, 1 Jun 2021 17:16:59 GMT, Patrick Concannon <pconcannon at openjdk.org> wrote:
>> src/java.base/share/classes/java/nio/file/attribute/FileTime.java line 256:
>>
>>> 254: nanos = (int) Math.floorMod(value, NANOS_PER_SECOND);
>>> 255: }
>>> 256: default -> throw new AssertionError("Unit not handled");
>>
>> The formatting here is all messed up, I think you'll need to fix this up before it can be reviewed.
>
> Sorry about that. I've changed it now. See 2f179b5
This still looks a bit messy because you've got 3 different styles in the one switch statement. It's okay to drop FileTime from the patch if you want as it's not worth spending time on.
-------------
PR: https://git.openjdk.java.net/jdk/pull/4285
More information about the nio-dev
mailing list