You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by kk...@apache.org on 2013/12/20 12:16:52 UTC

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

Author: kkolinko
Date: Fri Dec 20 11:16:51 2013
New Revision: 1552596

URL: http://svn.apache.org/r1552596
Log:
vote

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=1552596&r1=1552595&r2=1552596&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Fri Dec 20 11:16:51 2013
@@ -82,6 +82,20 @@ PATCHES PROPOSED TO BACKPORT:
   compare 6.0.x code with 7.0.x code.
   http://people.apache.org/~markt/patches/2013-12-19-xml-prep-part1-tc6-v1.patch
   +1: markt
+  +0: kkolinko:
+      I do not like that you change the values of validation flags when
+      STRICT_SERVLET_COMPLIANCE is set. This was a feature required for
+      Servlet 3.0, but I do not remember the same requirement for Servlet
+      2.5.
+      I expect that with this change the bug #351040 in Eclipse IDE will
+      become visible with Tomcat 6, while it is currently observed with
+      Tomcat 7 only. It is not our policy to provide workarounds for 3rd
+      party bugs, so this is not a veto.
+        https://bugs.eclipse.org/bugs/show_bug.cgi?id=351040
+      In ParserUtils:
+       // TODO Add a zero-arg constructor
+       This patch has such constructor, annotated as @Deprecated.
+      The rest of the patch is OK.
   -1:
 
 * Back-port some XML processing improvements (part 2)



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