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 2013/12/20 12:58:22 UTC
svn commit: r1552602 - /tomcat/tc6.0.x/trunk/STATUS.txt
Author: markt
Date: Fri Dec 20 11:58:21 2013
New Revision: 1552602
URL: http://svn.apache.org/r1552602
Log:
Update the proposal with a new patch.
Remove the comments that refer to the issue (completed TODO should have been removed) that has been resolved in the new patch
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=1552602&r1=1552601&r2=1552602&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Fri Dec 20 11:58:21 2013
@@ -80,7 +80,7 @@ PATCHES PROPOSED TO BACKPORT:
The patch could have been smaller but I opted to align some of the code with
Tomcat 7 (which meant re-ordering methods in places) to make it easier to
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
+ http://people.apache.org/~markt/patches/2013-12-20-xml-prep-part1-tc6-v2.patch
+1: markt
+0: kkolinko:
I do not like that you change the values of validation flags when
@@ -92,15 +92,11 @@ PATCHES PROPOSED TO BACKPORT:
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.
markt:
Servlet 2.5, SRV.14.4.3 requires validation for a J2EE compliant
implementation. The Eclipse bug is annoying but easy to workaround and
only present when using STRICT_SERVLET_COMPLIANCE.
- I'll fix the TODO I should have removed from ParserUtils.
-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