+1 for the review with a preference for Vadim’s suggestion. Brian
On Aug 1, 2016, at 10:15 AM, Phil Race <philip.r...@oracle.com> wrote: > That would be fine too. The main thing was getting rid of Class.forName > > -phil. > > On 08/01/2016 10:19 AM, Vadim Pakhnushev wrote: >> Why not just registry.getServiceProviders(ImageReaderSpi.class, ? >> >> Vadim >> >> On 01.08.2016 19:13, Jayathirth D V wrote: >>> Hi, >>> >>> Please review the following fix in JDK9 at your convenience : >>> >>> Bug :https://bugs.openjdk.java.net/browse/JDK-8160736 >>> Webrev :http://cr.openjdk.java.net/~jdv/8160736/webrev.00/ >>> >>> Root Cause : KSS tool has detected usage of class.forName where it can be >>> avoided. >>> Solution : Use <Classname.class> instead of using class.forName. >>> >>> Thanks, >>> Jay >> > >
