https://issues.apache.org/bugzilla/show_bug.cgi?id=51195

--- Comment #5 from Mark Thomas <ma...@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-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to