RFR: 8344332: (bf) Migrate DirectByteBuffer to use java.lang.ref.Cleaner [v11]
Alan Bateman
alanb at openjdk.org
Thu Jan 30 08:40:02 UTC 2025
On Wed, 29 Jan 2025 23:52:40 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
>> Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Revert waitForReferenceProcessing removals, see JDK-8305186
>
> src/java.base/share/classes/java/nio/Bits.java line 146:
>
>> 144: }
>> 145:
>> 146: if (canary == null || canary.isDead()) {
>
> If we're keeping Reference.waitForPendingReferences, why not continue to use it, rather than introducing
> the canary as a new, ad hoc, reference processing progress detector?
@kimbarrett Do you have a change coming to allow waitForPendingReferences be used by WB? I assume this will at least add a comment to the method (or whatever it changes to) to make it clear that it's for testing.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22165#discussion_r1935209408
More information about the nio-dev
mailing list