8219197: ThreadGroup.enumerate() may return wrong value
Langer, Christoph
christoph.langer at sap.com
Wed Mar 13 12:44:47 UTC 2019
Hi Daniel,
the fix looks good to me.
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?
Best regards
Christoph
> -----Original Message-----
> From: core-libs-dev <core-libs-dev-bounces at openjdk.java.net> On Behalf
> Of Daniel Fuchs
> Sent: Dienstag, 12. März 2019 16:59
> To: core-libs-dev <core-libs-dev at openjdk.java.net>
> Subject: RFR: 8219197: ThreadGroup.enumerate() may return wrong value
>
> Hi,
>
> Please find below a simple fix for
> 8219197: ThreadGroup.enumerate() may return wrong value
>
> http://cr.openjdk.java.net/~dfuchs/webrev_8219197/webrev.00/
>
> This is a bug in the implementation of the recursion,
> as enumerate(list, n, recurse) should never have returned
> a value < n.
>
> The test passes with the fix and fails 'often enough' without
> it (~ 1 times out of 2 or 3).
>
> best regards,
>
> -- daniel
More information about the core-libs-dev
mailing list