Author: markt
Date: Wed Feb 24 00:02:01 2010
New Revision: 915601

URL: http://svn.apache.org/viewvc?rev=915601&view=rev
Log:
Withdraw my patch - vote for Konstantin's

Modified:
    tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=915601&r1=915600&r2=915601&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Feb 24 00:02:01 2010
@@ -120,20 +120,12 @@
 
 
 * Memory leak detection for JMX and manager
-  
http://people.apache.org/~markt/patches/2010-02-12-memory-leak-detection.patch
-  +1: markt
-  -1: kkolinko:
-    - It does not compile, because WebappClassLoader.isStarted() was absent.
-    - /findleaks was not mapped to ManagerServlet
-    Updated patch proposed below.
-
-  Alternative proposal:
   
http://people.apache.org/~kkolinko/patches/2010-02-16_tc6_memory-leak-detection.patch
   It is corrected markt's patch
   (Added revs.909204,910584,910612
    TC 6 specifics: /findleaks should be explicitly mapped in web.xml,
    where TC 7 uses /text/findleaks, and that URL is mentioned in 
manager-howto.xml)
-  +1: kkolinko, fhanik
+  +1: kkolinko, fhanik, markt
   -1:
 
   kkolinko: Some further thoughts on naming/messages/documentation



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

Reply via email to