RFR: 8287107: CgroupSubsystemFactory.setCgroupV2Path asserts with freezer controller [v2]
Severin Gehwolf
sgehwolf at openjdk.java.net
Wed May 25 08:43:50 UTC 2022
On Tue, 24 May 2022 19:49:35 GMT, Ioi Lam <iklam at openjdk.org> wrote:
>> 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.
It confused me, fwiw. Anyway up to you. It's not super important.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8858
More information about the core-libs-dev
mailing list