Hi Simon, 3.0RC4 The testsuite doesn't hang, and it passes a simple test that failed on the previous version (I'll add it to the testsuite this weekend + other more complicated tests). Thanks for applying this.
I think there is still a problem with the ordering for findClass(String) (and possibly getURLs() - but not very likely) It will lock "this" before the reentrantLock which is the opposite of loadClass(String, boolean, ClassLoader). Also is synchronizing on "lock" necessary? It is already synchronized on the rentrantLock for these private methods. Is there a reason why these methods can't be inlined to save on stack opertions? Regards, Adrian >From: "Bordet, Simone" <[EMAIL PROTECTED]> >Reply-To: [EMAIL PROTECTED] >To: "JBoss Development Mailing List (E-mail)" ><[EMAIL PROTECTED]> >Subject: [JBoss-dev] UnifiedLoaderRepository deadlocks >Date: Thu, 30 May 2002 18:12:06 +0200 > >Hi, > >I have updated the ULR in HEAD to incorporate latest Adrian Brock's >suggestions (thanks Adrian), and made the relevant methods single threaded. >Adrian, if you can check if the changes I made correctly implement your >suggestions, will be great. > >Cheers > >Simon > >_______________________________________________________________ > >Don't miss the 2002 Sprint PCS Application Developer's Conference >August 25-28 in Las Vegas -- http://devcon.sprintpcs.com/adp/index.cfm > >_______________________________________________ >Jboss-development mailing list >[EMAIL PROTECTED] >https://lists.sourceforge.net/lists/listinfo/jboss-development _________________________________________________________________ Join the world’s largest e-mail service with MSN Hotmail. http://www.hotmail.com _______________________________________________________________ Don't miss the 2002 Sprint PCS Application Developer's Conference August 25-28 in Las Vegas -- http://devcon.sprintpcs.com/adp/index.cfm _______________________________________________ Jboss-development mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/jboss-development
