[11] RFR JDK-8198249: Remove deprecated Runtime::runFinalizersOnExit and System::runFinalizersOnExit
Peter Levart
peter.levart at gmail.com
Fri Feb 16 14:52:06 UTC 2018
Hi Mandy,
On 02/16/2018 05:09 AM, mandy chung wrote:
>
>
> On 2/15/18 7:29 PM, Martin Buchholz wrote:
>> This also changes the handling of (undeprecated) method
>> runFinalization and that's obviously related but could be a separate
>> changeset.
>>
>
> I agree I should separate the runFinalization change. I leave it in
> this webrev for now. I will send out the final webrevs before I push
> (I am feeling under the weather a bit now).
>
>> We'll need to merge with my own pending change to Finalizer.java.
>
> Merged. New version:
> http://cr.openjdk.java.net/~mchung/jdk11/webrevs/8198249/webrev.02/
>
> Mandy
I think that the following now never applies (in Finalizer) since
ReferenceQueue never returns the same Reference object more than once:
71 if (this.next == this) // already finalized
72 return;
You could change it into an assert to be comfortable. What do you think?
Regards, Peter
More information about the core-libs-dev
mailing list