On Fri, 9 Oct 2020 09:46:55 GMT, Severin Gehwolf <sgehw...@openjdk.org> wrote:
>> src/java.base/linux/classes/jdk/internal/platform/CgroupSubsystemFactory.java >> line 183: >> >>> 181: } >>> 182: } >>> 183: return relevantControllerFound; >> >> If I am reading the code right, then `relevantControllerFound` can be >> replaced with just `return true` or `return >> false`? > > I don't think so. It starts out as `false` and is being set to true on lines > 180 and 174. So we could change it to > `return false` here iff lines 174 and 180 would `return true`. Yes, that's what I meant: instead of dragging the boolean flag, do the early `return true`, or `return false` otherwise. Your choice if you want to make that change. ------------- PR: https://git.openjdk.java.net/jdk/pull/485
