harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Archie Cobbs <arc...@dellroad.org>
Subject Re: [jchevm+classlibadapter] Running classlib tests
Date Mon, 12 Jun 2006 01:35:57 GMT
Ivan Volosyuk wrote:
> I see change was made to String.intern() method. IMHO, the new code
> will not work correctly. I wanted to make WeakHashMap in which
> WeakReference to String holds WeakReference to that string. Current
> update changed it into WeakReference to String -> StrongReference to
> String. The code will prevent dropping outdated string from the
> WeakHashMap, as we have strong reference to that string in the
> WeakHashMap value.

You're right, that wasy my fault. Revision 413558 should fix it
(back to the original algorithm).

-Archie

__________________________________________________________________________
Archie Cobbs      *        CTO, Awarix        *      http://www.awarix.com

---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org


Mime
View raw message