I have a comment, that I'll bring out to the dev list -
Is there any problem with some other thread adding the entry for
currentURL while the if (jf==null) block is doing it's work? That's
one difference I can see between sync-ing the block, vs just syncing
the map.
I don't know if it matters, though.
geir
On Jan 30, 2007, at 7:48 AM, Mikhail Markov wrote:
Elena, thanks for the patch!
I've reviewed it and posted the comment to this JIRA.
Regards,
Mikhail
On 1/30/07, Elena Semukhina <[EMAIL PROTECTED]> wrote:
The DRLVM smoke test classloader.StressLoader fails intermittently.
Investigation revealed that java.net.URLClassLoader implementation
does
not
take care of synchronizing on access to its IdentityHashMap field. I
suggested a patch to fix the issue at
https://issues.apache.org/jira/browse/HARMONY-2982.
Could anyone please take a look and review the patch?
--
Thanks,
Elena
