Re: [doc]Small modification on the WeakHashMap doc

Thanks David. Thanks Mike. The patch has been committed. On 06/04/2012 11:33 AM, Mike Duigou wrote: 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 Ala

Re: [doc]Small modification on the WeakHashMap doc

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, >>>

Re: [doc]Small modification on the WeakHashMap doc

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 w

Re: [doc]Small modification on the WeakHashMap doc

Thanks David. Do I need another review? 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

Re: [doc]Small modification on the WeakHashMap doc

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

Re: [doc]Small modification on the WeakHashMap doc

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/ On 05/07/2012

Re: [doc]Small modification on the WeakHashMap doc

Thanks David :-) 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 re

Re: [doc]Small modification on the WeakHashMap doc

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

Re: [doc]Small modification on the WeakHashMap doc

Hi guys, Does anyone interested in this issue? 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))

[doc]Small modification on the WeakHashMap doc

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. B