You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@tomcat.apache.org by ma...@apache.org on 2007/11/14 00:19:12 UTC

svn commit: r594704 - /tomcat/tc6.0.x/trunk/STATUS

Author: markt
Date: Tue Nov 13 15:19:10 2007
New Revision: 594704

URL: http://svn.apache.org/viewvc?rev=594704&view=rev
Log:
New patch in response to -1.
Wasn't sure whether to leave the original patch and votes for completeness - decided clarity was better and the history is always available in svn.

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

Modified: tomcat/tc6.0.x/trunk/STATUS
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS?rev=594704&r1=594703&r2=594704&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS (original)
+++ tomcat/tc6.0.x/trunk/STATUS Tue Nov 13 15:19:10 2007
@@ -42,11 +42,6 @@
   +1: markt, remm
   -1:
 
-* Fix BZ 43675 - Possible logging related classloader leak
-  http://people.apache.org/~markt/patches/2007-10-30-Bug43675.patch
-  +1: markt
-  -1: remm (in ServerCookie, log(String) should probably be removed)
-
 * Fix licensing of JSP 2.1 schema
   svn diff -c r593814
   +1: markt, remm, jfclere
@@ -60,4 +55,11 @@
 * Add tests for the cookie parsing.
   http://people.apache.org/~jfclere/patches/test_cookies.patch
   +1: jfclere
+  -1:
+
+* Fix BZ 43675 - Possible logging related classloader leak
+  Better patch that removes ServerCookie.log(String) entirely as per
+  Remy's -1 comment on previous patch
+  http://svn.apache.org/viewvc?view=rev&revision=594700
+  +1: markt
   -1:



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