Daniel, > On 13 Mar 2019, at 12:59, Daniel Fuchs <daniel.fu...@oracle.com> wrote: > > Hi Christoph, > > On 13/03/2019 12:44, Langer, Christoph wrote: >> In the test, you could remove the import java.util.Set; in line 36 and >> List<ThreadGroup> groups of line 74 is not needed as well. >> BTW, as you are touching it, >> src/java.base/share/classes/java/lang/ThreadGroup.java does not need the >> import jdk.internal.misc.VM; in line 30 neither. Maybe you want to clean >> that up, too? > > Good points! Done. > http://cr.openjdk.java.net/~dfuchs/webrev_8219197/webrev.01/ > <http://cr.openjdk.java.net/~dfuchs/webrev_8219197/webrev.01/>
This change looks good to me. -Chris.