Review Request: JDK-8235521: Replacement API for Unsafe::ensureClassInitialized

Mandy Chung mandy.chung at oracle.com
Thu Jun 4 20:16:41 UTC 2020


On 6/4/20 12:39 PM, mark.reinhold at oracle.com wrote:
> I agree that the `ensure` prefix is useful here.
>
> This method can only ensure that a class is initialized, so including
> `Class` in the method name seems redundant.  I’d go with the shorter
> `ensureInitialized`.
>
> - Mark

Thanks for the advice.   I go with the shorter name.

I finalized an updated CSR per all feedbacks which includes the 
deprecation `sun.misc.Unsafe::shouldBeInitialized`.
    https://bugs.openjdk.java.net/browse/JDK-8245871

updated webrev:
http://cr.openjdk.java.net/~mchung/jdk15/webrevs/8235521/webrev.02

thanks
Mandy


More information about the core-libs-dev mailing list