Konstantin Kolinko <[email protected]> wrote:

>2012/9/24  <[email protected]>:
>> Author: markt
>> Date: Sun Sep 23 21:09:22 2012
>> New Revision: 1389145
>>
>> URL: http://svn.apache.org/viewvc?rev=1389145&view=rev
>> Log:
>> Some more low(ish) hanging fruit from the allocation hit list. This
>accounts for ~8% due to the way Thread stores names.


>-1 for ReadWriteLock.
>The problem with WeakHashMap is that its get() operation can modify it
>(as it processes referencequeue).  Thus you cannot use ReadWriteLock,
>but must use plain synchronization here.

Yep. I'll remove that class. ThreadLocal is a better solution for this problem 
anyway. Not sure why I didn't think of that the first time. I'll fix this later 
today.

Cheers,

Mark


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to