[cp-patches] FYI: Using a WeakIdentityHashMap for thread locals

[Jeroen Frijters]

address@hidden wrote:
> >>>>> "Jeroen" == Jeroen Frijters <address@hidden> writes:
> 
> Jeroen> I propose the attached change to the handling of 
> thread locals. This
> Jeroen> fixes bug #24858 and makes the code slightly more 
> obvious (at the cost
> Jeroen> of introducing the new class WeakIdentityHashMap).
> 
> Jeroen> Comments?
> 
> IMO it is fine.

Thanks for reviewing. I've committed the patch.

Regards,
Jeroen

2006-01-06  Jeroen Frijters  <address@hidden>

        PR classpath/24858
        * gnu/java/util/WeakIdentityHashMap.java: New file.
        * java/lang/InheritableThreadLocal.java
        (newChildThread): Modified to remove key indirection.
        * java/lang/Thread.java
        (locals): Changed type to WeakIdentityHashMap.
        (getThreadLocals): Instantiate WeakIdentityHashMap instead of
        WeakHashMap.
        * java/lang/ThreadLocal.java
        (key, Key): Removed.
        (get, set): Changed to use "this" instead of "key".

threadlocal.patch.txt
Description: threadlocal.patch.txt

WeakIdentityHashMap.java
Description: WeakIdentityHashMap.java