[doc]Small modification on the WeakHashMap doc

[Mike Duigou]

The change looks good to me.
On Jun 3 2012, at 20:05 , David Holmes wrote:

> On 4/06/2012 11:55 AM, Charles Lee wrote:
>> Thanks David. Do I need another review?
> 
> Yes. Someone from TL - Mike or Alan most likely.
> 
> David
> 
>> On 06/03/2012 06:15 AM, David Holmes wrote:
>>> Hi Charles,
>>> 
>>> I have no problem with this clarification in the implementation notes
>>> being added. I've checked with Joe and it does not require CCC approval.
>>> 
>>> David
>>> -----
>>> 
>>> On 28/05/2012 5:36 PM, Charles Lee wrote:
>>>> 
>>>> Hi devs,
>>>> 
>>>> I'd like to propose a new minor change for the WeakHashMap doc, which I
>>>> got it from David :-)
>>>> 
>>>> Would anyone got some time to take a look this fix[1]?
>>>> 
>>>> 1. http://cr.openjdk.java.net/~littlee/7166055/webrev.01/
>>>> <http://cr.openjdk.java.net/%7Elittlee/7166055/webrev.01/>
>>>> 
>>>> On 05/07/2012 11:45 AM, David Holmes wrote:
>>>>> Hi Charles,
>>>>> 
>>>>> On 7/05/2012 1:05 PM, Charles Lee wrote:
>>>>>> Does anyone interested in this issue?
>>>>> 
>>>>> Interest and time are two different things :)
>>>>> 
>>>>> A shorter form would be:
>>>>> 
>>>>> "If the values in the map do not rely on the map holding strong
>>>>> references to them, then one way to deal with this is ...
>>>>> 
>>>>> David
>>>>> 
>>>>>> On 05/03/2012 02:52 PM, Charles Lee wrote:
>>>>>>> Hi guys,
>>>>>>> 
>>>>>>> In the Implementation notes of WeakHashMap[1], says:
>>>>>>> 
>>>>>>> /One way to deal with this is to wrap values themselves within
>>>>>>> WeakReferences before inserting, as in: m.put(key, new
>>>>>>> WeakReference(value)), and then unwrapping upon each get./
>>>>>>> 
>>>>>>> However, it is not concise and a little misleading. Because the value
>>>>>>> in the WeakReference can be GC'd if there are no strong reference to
>>>>>>> it. This behaviour surprises some customers.
>>>>>>> How about add a statement like [2]:
>>>>>>> 
>>>>>>> /However, as the use of WeakReference in this manner will not prevent
>>>>>>> value objects from being GC'd, this approach is only useful when
>>>>>>> entries in the map are not relied upon for keeping the underlying
>>>>>>> value objects "live"./
>>>>>>> 
>>>>>>> 
>>>>>>> 
>>>>>>> 
>>>>>>> [1]:
>>>>>>> http://docs.oracle.com/javase/7/docs/api/java/util/WeakHashMap.html
>>>>>>> [2]: http://cr.openjdk.java.net/~littlee/7166055/webrev.00/
>>>>>>> <http://cr.openjdk.java.net/%7Elittlee/7166055/webrev.00/>
>>>>>>> 
>>>>>> 
>>>>>> 
>>>>> 
>>>> 
>>>> 
>>>> --
>>>> Yours Charles
>>>> 
>>> 
>> 
>>