tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Konstantin Kolinko <knst.koli...@gmail.com>
Subject Re: svn commit: r907538 - /tomcat/trunk/java/org/apache/jasper/compiler/Parser.java
Date Mon, 08 Feb 2010 12:49:26 GMT
2010/2/8  <markt@apache.org>:
> Author: markt
> Date: Mon Feb  8 02:00:58 2010
> New Revision: 907538
>
> URL: http://svn.apache.org/viewvc?rev=907538&view=rev
> Log:
> Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48668
> Don't look for EL matches (which swallow the EL match string) if EL is not enabled
>
> Modified:
>    tomcat/trunk/java/org/apache/jasper/compiler/Parser.java
>

> -        } else if (reader.matches("${") && !pageInfo.isELIgnored()) {
> +        } else if (!pageInfo.isELIgnored() && reader.matches("${")) {
>             parseELExpression(parent, '$');
> -        } else if (reader.matches("#{") && !pageInfo.isELIgnored()) {
> +        } else if (!pageInfo.isELIgnored() && reader.matches("#{")) {
>             parseELExpression(parent, '#');
>         } else if (reader.matches("<jsp:")) {

For the "#{" expression the deferredSyntaxAllowedAsLiteral option
should be taken care as well.

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


Mime
View raw message