tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mark Thomas <ma...@apache.org>
Subject Re: svn commit: r603074 - /tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java
Date Tue, 11 Dec 2007 19:32:59 GMT
Filip Hanik - Dev Lists wrote:
> that's a good point, one would expect most names to be pooled when it
> comes to implicit class loading, although as you say, not a guarantee.
> in findClassInternal, we do synchronized(this), so maybe we should not
> need to introduce another locking object, when we already have the
> instance itself.

You are right. I was trying to be clever and failing miserably ;). I'll
update the patch.

Mark



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


Mime
View raw message