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.

Reply via email to