> On 9 Feb 2019, at 19:25, Martin Buchholz <marti...@google.com> wrote: > > Thanks, Chris. > > On Sat, Feb 9, 2019 at 6:34 AM Chris Hegarty <chris.hega...@oracle.com > <mailto:chris.hega...@oracle.com>> wrote: >> 8215359: InnocuousForkJoinWorkerThread.setContextClassLoader needlessly >> throws >> https://cr.openjdk.java.net/~martin/webrevs/jdk/jsr166-integration/InnocuousForkJoinWorkerThread/index.html >> >> <https://cr.openjdk.java.net/~martin/webrevs/jdk/jsr166-integration/InnocuousForkJoinWorkerThread/index.html> > > Looks good. A positive test for setContextClassLoader(null) could be > added. > > good point - let's improve the test: >
The updated test looks good. -Chris.
