You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ti...@apache.org on 2010/10/03 22:27:12 UTC

svn commit: r1004039 - in /tomcat/tc6.0.x/trunk: STATUS.txt java/org/apache/jasper/compiler/ParserController.java webapps/docs/changelog.xml

Author: timw
Date: Sun Oct  3 20:27:12 2010
New Revision: 1004039

URL: http://svn.apache.org/viewvc?rev=1004039&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49998
Handle single quoted attributes in detection of jsp:root element in XML syntax JSP files.

Modified:
    tomcat/tc6.0.x/trunk/STATUS.txt
    tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/ParserController.java
    tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1004039&r1=1004038&r2=1004039&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Sun Oct  3 20:27:12 2010
@@ -252,9 +252,3 @@ PATCHES PROPOSED TO BACKPORT:
   http://svn.apache.org/viewvc?rev=1003861&view=rev
   +1: markt, timw, kkolinko
   -1:
-
-* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49998
-  Handle single quoted attributes in detection of jsp:root element in XML syntax JSP files.
-  http://svn.apache.org/viewvc?view=revision&revision=1003923
-  +1: timw, kkolinko, markt
-  -1:

Modified: tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/ParserController.java
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/ParserController.java?rev=1004039&r1=1004038&r2=1004039&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/ParserController.java (original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/ParserController.java Sun Oct  3 20:27:12 2010
@@ -612,10 +612,12 @@ class ParserController implements TagCon
                     && Character.isWhitespace(root.charAt(index))) {
                 index++;
             }
-            if (index < root.length() && root.charAt(index++) == '"'
-                && root.regionMatches(index, JSP_URI, 0,
-                        JSP_URI.length())) {
-                return true;
+            if (index < root.length() 
+                && (root.charAt(index) == '"' || root.charAt(index) == '\'')) {
+                index++;
+                if (root.regionMatches(index, JSP_URI, 0, JSP_URI.length())) {
+                    return true;
+                }
             }
         }
 

Modified: tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?rev=1004039&r1=1004038&r2=1004039&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Sun Oct  3 20:27:12 2010
@@ -145,6 +145,15 @@
       </fix>
     </changelog>
   </subsection>
+  <subsection name="Jasper">
+    <changelog>
+      <fix>
+        <bug>49987</bug>: Make jsp:root detection work with single quoted
+        attributes as well. (timw)
+      </fix>
+    </changelog>
+  </subsection>
+  
   <subsection name="Other">
     <changelog>
       <fix>



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