svn commit: r603340 - /tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java

[markt]

Author: markt
Date: Tue Dec 11 11:36:00 2007
New Revision: 603340

URL: http://svn.apache.org/viewvc?rev=603340&view=rev
Log:
Better patch based on Filip's comments

Modified:
    tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java

Modified: tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java?rev=603340&r1=603339&r2=603340&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java Tue Dec 
11 11:36:00 2007
@@ -167,12 +167,6 @@
      */
     boolean antiJARLocking = false; 
     
-    /**
-     * Lock to prevent attempts to load duplicate classes from external
-     * repositories.
-     */
-    private Object lock = new Object();
-
     // ----------------------------------------------------------- Constructors
 
 
@@ -888,7 +882,7 @@
             }
             if ((clazz == null) && hasExternalRepositories) {
                 try {
-                    synchronized (lock) {
+                    synchronized (this) {
                         clazz = super.findClass(name);
                     }
                 } catch(AccessControlException ace) {



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]