On Fri, 9 Oct 2020 09:23:22 GMT, Aleksey Shipilev <[email protected]> wrote:
>> Severin Gehwolf has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Return false in case of no pattern match > > 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`. ------------- PR: https://git.openjdk.java.net/jdk/pull/485
