tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 51195] "Find leaks" reports a false positive memory/classloader leak
Date Fri, 20 May 2011 18:58:43 GMT
https://issues.apache.org/bugzilla/show_bug.cgi?id=51195

--- Comment #5 from Mark Thomas <markt@apache.org> 2011-05-20 18:58:43 UTC ---
A couple of comments on the proposed patch.

1. This looks like clean-up code and as such is better placed in the
WebappClassLoader as another ClearReferencesXXX method.

2. Rather than clearing the entire map, it would be better to iterate over the
map and remove only those entries that had been loaded by the class
WebappClassLoader currently being stopped.

In terms of checking the build, "ant" should suffice rather than the full "ant
release". You;ll still get a working Tomcat instance you can test in
output/build.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

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


Mime
View raw message