Konstantin Kolinko <knst.koli...@gmail.com> wrote:

>2012/9/24  <ma...@apache.org>:
>> 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: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to