<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 17, 2015 at 9:34 PM, Mandy Chung <span dir="ltr"><<a href="mailto:mandy.chung@oracle.com" target="_blank">mandy.chung@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
> On Dec 17, 2015, at 6:05 PM, Tao Mao <<a href="mailto:yiyeguhu@gmail.com">yiyeguhu@gmail.com</a>> wrote:<br>
><br>
> Hi Kim,<br>
><br>
> 34  * <p> Suppose the garbage collector determines at a certain point in time<br>
>   35  * that an object is <a href="package-summary.html#reachability"><br>
>   36  * phantom reachable</a>.  At that time it will atomically clear<br>
>   37  * all phantom references to that object and all phantom references to<br>
>   38  * any other phantom-reachable objects from which that object is reachable.<br>
>   39  * At the same time or at some later time it will enqueue those newly-cleared<br>
>   40  * phantom references that are registered with reference queues.<br>
><br>
> XYZRefenrece concept is never being understood too well besides the authors who spend time grinding words. WeakReference (<a href="https://docs.oracle.com/javase/7/docs/api/java/lang/ref/WeakReference.html" rel="noreferrer" target="_blank">https://docs.oracle.com/javase/7/docs/api/java/lang/ref/WeakReference.html</a>) states this: "At that time it will atomically clear all weak references to that object and all weak references to any other weakly-reachable objects from which that object is reachable through a chain of strong and soft references."<br>
><br>
> On line 38, can we also add "through a chain of strong, soft, and weak references." to the above, which I think is the case?<br>
><br>
<br>
</span>Yes it is and this is not strictly needed. For phantom reference case, this can be left “unqualified” as it’s basically a chain of any reference type.<br></blockquote><div><br></div><div>Strictly speaking, any reference type but phantom reference. </div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> In addition, in src/java.base/share/classes/java/lang/ref/package-info.java, can we add WeakHashMap (<a href="https://docs.oracle.com/javase/7/docs/api/java/util/WeakHashMap.html" rel="noreferrer" target="_blank">https://docs.oracle.com/javase/7/docs/api/java/util/WeakHashMap.html</a>) as a concrete example of so-called canonicalized mappings to help readers understand its usage? I always need to look up for what "canonicalized mappings" really means and end up with "OK, it's WeakHashMap”.<br>
<br>
</span>In the Notification section, it does reference WeakHashMap (although it might not contain the exact description you are looking at).<br></blockquote><div><br></div><div>Good, I see it.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
Mandy<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> Thanks.<br>
> Tao Mao<br>
><br>
><br>
><br>
> On Thu, Dec 17, 2015 at 1:30 PM, Kim Barrett <<a href="mailto:kim.barrett@oracle.com">kim.barrett@oracle.com</a>> wrote:<br>
> On Dec 4, 2015, at 1:07 PM, Kim Barrett <<a href="mailto:kim.barrett@oracle.com">kim.barrett@oracle.com</a>> wrote:<br>
> ><br>
> > On Dec 3, 2015, at 6:04 PM, Mandy Chung <<a href="mailto:mandy.chung@oracle.com">mandy.chung@oracle.com</a>> wrote:<br>
> >><br>
> >>> [Indeed, this whole section isn't strictly necessary; all of it can be<br>
> >>> inferred from information in other places.]<br>
> >><br>
> >> Agree.  This section is no longer necessary and maybe just remove it:<br>
> ><br>
> > I wasn't actually intending to suggest removal.  It seems like there<br>
> > is useful overview information to be had here, rather than requiring<br>
> > readers to make not necessarily obvious inferences.  My impression is<br>
> > that readability is valued more highly than terseness in Java<br>
> > documentation.<br>
><br>
> Sorry for the long gap in the discussion.  Mandy and I have been<br>
> talking about what to do about the "Automatically-cleared references"<br>
> section that was the topic of some debate.  We've decided to eliminate<br>
> it, which led us to some additional modifications.<br>
><br>
> Here's the latest set of webrevs:<br>
> <a href="http://cr.openjdk.java.net/~kbarrett/8071507/jdk.06/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~kbarrett/8071507/jdk.06/</a><br>
> <a href="http://cr.openjdk.java.net/~kbarrett/8071507/hotspot.06/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~kbarrett/8071507/hotspot.06/</a><br>
><br>
> They've been rebased to hs-rt tip, which required resolving a minor<br>
> merge conflict with a nearby change to logging in<br>
> referenceProcessor.cpp.  Other than that, there are only specification<br>
> wording changes.  Here's the incremental change from the previous<br>
> version:<br>
> <a href="http://cr.openjdk.java.net/~kbarrett/8071507/jdk.06.inc/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~kbarrett/8071507/jdk.06.inc/</a><br>
><br>
> We are treating the discussions around changing PhantomReference to<br>
> forbid a null queue as out of scope for this change.  As Mandy said<br>
> earlier, she might propose a separate change for that in the future.<br>
><br>
><br>
<br>
</div></div></blockquote></div><br></div></div>