tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jan Luehe <Jan.Lu...@Sun.COM>
Subject Re: cvs commit: jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/core StandardContext.java
Date Thu, 17 Feb 2005 18:33:17 GMT


Remy Maucherat wrote:
> luehe@apache.org wrote:
> 

>>  This patch should fix the issue.
>>  
>>  Let me know if you see any problems.
> 
> 
> There's the same code (and more) in WebappClassLoader.stop():
> 
>          // Clear the classloader reference in common-logging
>          org.apache.commons.logging.LogFactory.release(this);
>          // Clear the classloader reference in the VM's bean introspector
>          java.beans.Introspector.flushCaches();
> 
> IMO, it's the right place to do this kind of cleaning up in the 
> classloader, since it's CL related.

You're absolutely right. You had fixed this in TC long time ago
(revision 1.20). This commit did slip through the cracks, so our
internal version was still suffering from this leak.

Thanks!

Jan



---------------------------------------------------------------------
To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org


Mime
View raw message