2010/1/12 <ma...@apache.org>:
> Author: markt
> Date: Tue Jan 12 00:10:46 2010
> New Revision: 898126
>
> URL: http://svn.apache.org/viewvc?rev=898126&view=rev
> Log:
> Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=47977
> Using a body with tags specified to have empty body content should cause an
> error
>
> Modified:
> tomcat/trunk/java/org/apache/jasper/compiler/JspDocumentParser.java
>
> @@ -426,9 +430,10 @@
> if (scriptlessBodyNode == null
> &&
> bodyType.equalsIgnoreCase(TagInfo.BODY_CONTENT_SCRIPTLESS)) {
> scriptlessBodyNode = node;
> - }
> - else if
> (TagInfo.BODY_CONTENT_TAG_DEPENDENT.equalsIgnoreCase(bodyType)) {
> + } else if
> (TagInfo.BODY_CONTENT_TAG_DEPENDENT.equalsIgnoreCase(bodyType)) {
> tagDependentPending = true;
> + } else if (TagInfo.BODY_CONTENT_EMPTY.equals(bodyType)) {
> + tagEmptyBody = node;
> }
> }
> }
Why there is equalsIgnoreCase() in the old code? Did the specification
ever allow wrong-cased values there? If yes, then maybe the added
clause should also use equalsIgnoreCase.
Best regards,
Konstantin Kolinko
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
