RFR: 8287107: CgroupSubsystemFactory.setCgroupV2Path asserts with freezer controller [v2]

Ioi Lam iklam at openjdk.java.net
Tue May 24 21:13:05 UTC 2022


On Tue, 24 May 2022 19:34:16 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

>> This is not a fall-through because the previous line ends with a `break`.
>
> My bad. How about `Intentional incomplete switch. There are ...`? Anyway, why is the empty `default` case needed other than for the comment?

To me, the `default:` switch is a clear indication that "everything else comes here". So you won't be confused whether the comment is related to the last line just above the comment.

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

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


More information about the core-libs-dev mailing list