Author: kkolinko
Date: Sun Dec 20 04:40:16 2009
New Revision: 892560

URL: http://svn.apache.org/viewvc?rev=892560&view=rev
Log:
votes

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=892560&r1=892559&r2=892560&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Sun Dec 20 04:40:16 2009
@@ -311,9 +311,16 @@
 
 * Further improvement to memory leak protection - XML parsing
   http://svn.apache.org/viewvc?rev=884341&view=rev
-  +1: markt, rjung
+  +1: markt, rjung, kkolinko
   -1: 
   rjung: "memopry" -> "memory" in docs patch, "idenitfy" -> "identify" in 
comment
+  kkolinko: I thought that maybe DocumentBuilderFactory.newInstance()
+    should be inside try/catch as well. It can throw FactoryConfigurationError.
+    I do not really care, as it is only a question of providing a message.
+    If anybody encounters this, they can turn off the xmlParsingProtection
+    flag and go on.
+    In this sense, it could be catch(Throwable) here, and in other places
+    in that class as well.
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=47537
   Return an error page rather than a zero length 200 response if the forward to
@@ -351,7 +358,7 @@
   Ensure HTTP header values meet the requirements of RFC2616
   http://svn.apache.org/viewvc?rev=892293&view=rev
   +1: markt, rjung
-  -1: 
+  -1: kkolinko: see my reply to r892293 on dev@
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=47930
   Prevent parallel requests for the same session causing the PersistentManager
@@ -363,12 +370,12 @@
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48413
   Correct French translations
   http://svn.apache.org/viewvc?rev=892555&view=rev
-  +1: markt
+  +1: markt, kkolinko
   -1: 
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=47976
   Align usage message and javadoc with implementation
   http://svn.apache.org/viewvc?view=revision&revision=832214
   http://svn.apache.org/viewvc?view=revision&revision=892464
-  +1: markt
+  +1: markt, kkolinko
   -1: 



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

Reply via email to