Mailing-List: contact dev-help@tomcat.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Tomcat Developers List" <dev@tomcat.apache.org>
Received-SPF: neutral (nike.apache.org: local policy)
Message-ID: <4B472593.5000406@apache.org>
Date: Fri, 08 Jan 2010 12:31:15 +0000
From: Mark Thomas <markt@apache.org>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.2; en-GB;
 rv:1.9.1.5) Gecko/20091204 Thunderbird/3.0
MIME-Version: 1.0
To: Tomcat Developers List <dev@tomcat.apache.org>
Subject: Re: EL issues and 6.0.x release
References: <4B4718BF.5000208@apache.org> <4B47227B.6030603@apache.org>
In-Reply-To: <4B47227B.6030603@apache.org>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit

On 08/01/2010 12:18, Tim Funk wrote:
> +1 to trunk
> -0 for backport to 6.0.x
> 
> Is this legal? ${'\'}${1+1}
> Shouldn't it be: ${'\\'}${1+1}
Yes it should.

and the Java code to append the \ is:
result.append("${'\\\\'}");

This is part of what makes this so tricky to get right. The quoting
rules vary between Java, JSPs and EL. A number of the initial failures
with my tests were because I had quoted stuff incorrectly in the test case.

The good news is that one of my test cases was failing because of
exactly this error. As you'll see which my next commit (hopefully later
today) things are a lot easier to test and there are a lot more test
cases. In fact the majority of the changes will be test cases rather
than changes to Jasper/EL.

Mark


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