> On Sep 7, 2016, at 2:02 PM, Paul Sandoz <[email protected]> wrote: > > >> On 7 Sep 2016, at 12:42, Mandy Chung <[email protected]> wrote: >> >> Webrev: >> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8165563/webrev.00/ >> > > +1 > > Do we have any tests that can be cleaned up? If so log a separate issue for > that?
I did scan on the jdk/test and don’t see any test checking for null system class loader. Some of them actually assume non-null. On the other hand, there may be JDK classes that handle the null case and can be cleaned up. I didn’t do the audit and plan to leave it for the component teams to follow up. Mandy
