> On Nov 9, 2016, at 2:53 PM, Brent Christian <brent.christ...@oracle.com> > wrote: > > Hi, > > It seems that the method name used in 8165793[1], "isParallelCapable", was a > little *too* intuitive. An existing use of that method name has been > uncovered (via NetBeans, in Eclipse Equinox). > > Because the newly added method was marked final, the pre-existing method > results in a VerifyError under JDK 9 (for overriding a final method). > > Please review my fix, which makes ClassLoader.isParallelCapable() non-final. > The semantics of such a boolean-returning isXXXX method are clear - it's > unlikely that existing methods would use different semantics. > > I've included an @apiNote (ala ClassLoader.getName()) explaining that the > non-final-ness here is strictly for compatibility. > > Bug: > https://bugs.openjdk.java.net/browse/JDK-8169435 > Webrev: > http://cr.openjdk.java.net/~bchristi/8169435/webrev.0/
+1 Mandy