JDK-8027351: (ref) Base's class finalize method not invoked if a private finalize method exists in its subclass

Mandy Chung mandy.chung at oracle.com
Fri Nov 1 21:11:20 UTC 2013


On 11/1/13 1:37 PM, mark.reinhold at oracle.com wrote:
> 2013/11/1 4:15 -0700, mandy.chung at oracle.com:
>> http://cr.openjdk.java.net/~mchung/jdk8/webrevs/8027351/webrev.00/
> Looks good.
>
> Just one question: In Finalizer.java, at line 97 you look up the
> JavaLangAccess object every single time.  Is it worth caching that
> earlier, maybe when the finalize thread starts, since it will never
> change?

I was expecting that would get optimized during runtime and it's a 
simple getter method. It's a good suggestion to cache it at the finalize 
thread start time and here is the revised webrev:

http://cr.openjdk.java.net/~mchung/jdk8/webrevs/8027351/webrev.01/

Thanks for the review.
Mandy




More information about the core-libs-dev mailing list