Hi,

On 2016-11-09 23:53, Brent Christian 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/

looks good to me.

Thanks!

/Claes

Reply via email to