8219197: ThreadGroup.enumerate() may return wrong value

Chris Hegarty chris.hegarty at oracle.com
Wed Mar 13 13:59:53 UTC 2019


Daniel,

> On 13 Mar 2019, at 12:59, Daniel Fuchs <daniel.fuchs at 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.



More information about the core-libs-dev mailing list