RFR: JDK-8149925 We don't need jdk.internal.ref.Cleaner any more

Peter Levart peter.levart at gmail.com
Fri Apr 1 16:46:55 UTC 2016



On 04/01/2016 06:08 PM, Peter Levart wrote:
>
>
> On 04/01/2016 05:18 PM, Peter Levart wrote:
>> @Roger:
>>
>> ...
>>
>> About entanglement between nio Bits and 
>> ExtendedCleaner.retryWhileHelpingClean(). It is the same level of 
>> entanglement as between the DirectByteBuffer constructor and 
>> Cleaner.register(). In both occasions an action is provided to the 
>> Cleaner. Cleaner.register() takes a cleanup action and 
>> ExtendedCleaner.retryWhileHelpingClean() takes a retriable 
>> "allocating" or "reservation" action. "allocation" or "reservation" 
>> is the opposite of cleanup. Both methods are encapsulated in the same 
>> object because those two functions must be coordinated. So I think 
>> that collocating them together makes sense. What do you think?
>
> ...to illustrate what I mean, here's a variant that totally untangles 
> Bits from Cleaner and moves the whole Cleaner interaction into the 
> DirectByteBuffer itself:
>
> http://cr.openjdk.java.net/~plevart/jdk9-dev/removeInternalCleaner/webrev.13.part2/ 
>
>
> Notice the symmetry between Cleaner.retryWhileHelpingClean : 
> Cleaner.register and Allocator : Deallocator ?
>
>
> Regards, Peter
>

And here's also a diff between webrev.12.part2 and webrev.13.part2:

http://cr.openjdk.java.net/~plevart/jdk9-dev/removeInternalCleaner/webrev.diff.12to13.part2/

Regards, Peter




More information about the core-libs-dev mailing list