https://issues.apache.org/bugzilla/show_bug.cgi?id=52850
--- Comment #5 from Mark Thomas <ma...@apache.org> 2012-03-09 15:29:13 UTC --- (In reply to comment #0) > - Most of the detection and fixing code has been tested ONLY on Sun JVMs. Correct. > - Tests in tomcat7.source\test\org\apache\catalina\loader do not seem to run > successfully and are incomplete for all the protection that Tomcat provides > for > classloader memory leaks. The tests that are present do work. Yes, code coverage of the tests is a long way from 100%. > - For some categories of threadlocal memory leaks the key and value are not > displayed correctly in the warning messages; particularly ones dealing with > indirect references to threadlocals I don't see any changes to address this. Did I miss them? > - org.apache.catalina.loader.WebappClassLoader.loadedByThisOrChild(Object) > incorrectly traverses the object classloader hierarchy instead of the current > (this) classloader hierarchy No. The test is not "loaded by this or parent", it is "loaded by this or child" and is correct. I haven't looked at the tests yet. I'll commit the fixes (less the change to WebappClassLoader.loadedByThisOrChild(Object)) with the tests once I have reviewed the new tests. -- 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
