RFR: 8071507: (ref) Clear phantom reference as soft and weak references do

Mandy Chung mandy.chung at oracle.com
Fri Dec 4 21:58:55 UTC 2015


> On Dec 4, 2015, at 10:07 AM, Kim Barrett <kim.barrett at oracle.com> wrote:
> 
> On Dec 3, 2015, at 6:04 PM, Mandy Chung <mandy.chung at oracle.com> wrote:
>> 
>>> [Indeed, this whole section isn't strictly necessary; all of it can be
>>> inferred from information in other places.]
>> 
>> Agree.  This section is no longer necessary and maybe just remove it:
> 
> I wasn't actually intending to suggest removal.  It seems like there
> is useful overview information to be had here, rather than requiring
> readers to make not necessarily obvious inferences.  My impression is
> that readability is valued more highly than terseness in Java
> documentation.

Actually I considered to remove it in the past but just tried to minimize the spec change.  I believe one purpose of that section was intended to call out the difference that PhantomReference is not automatically-cleared that is no longer true.

In addition it’s clearly mentioned in PhantomReference constructor.  I don’t see the need for it in the package summary.  The reader will look at the PhantomRefererence to use it..

"It is possible to create a phantom reference with a null queue, but such a reference is completely useless: Its get method will always return null and, since it does not have a queue, it will never be enqueued.”

The puzzling part to me is why PhantomReference accepts null ReferenceQueue.   I can’t evaluate how high of the source incompatibility risk if we fixed it but I may propose that in a future release until I have cycle.

Mandy


More information about the core-libs-dev mailing list