Re: EL issues and 6.0.x release

2010-02-01 Thread Konstantin Kolinko
2010/1/31 Mark Thomas : > On 30/01/2010 17:41, Mark Thomas wrote: >> On 30/01/2010 07:33, Konstantin Kolinko wrote: >>> Regarding the implementation, AttributeParser.java class. I think >>> that, based on the above, we can fix it to solve bug 48627. Other >>> parts of the new implementation will re

Re: EL issues and 6.0.x release

2010-01-30 Thread Mark Thomas
On 30/01/2010 17:41, Mark Thomas wrote: > On 30/01/2010 07:33, Konstantin Kolinko wrote: >> Regarding the implementation, AttributeParser.java class. I think >> that, based on the above, we can fix it to solve bug 48627. Other >> parts of the new implementation will remain unchanged. > > I'll take

Re: EL issues and 6.0.x release

2010-01-30 Thread Mark Thomas
On 30/01/2010 07:33, Konstantin Kolinko wrote: > Regarding the implementation, AttributeParser.java class. I think > that, based on the above, we can fix it to solve bug 48627. Other > parts of the new implementation will remain unchanged. I'll take another look at this. I thought that this wouldn

Re: EL issues and 6.0.x release

2010-01-29 Thread Konstantin Kolinko
2010/1/8 Mark Thomas : > I think I have got to the bottom of why the EL has been so fragile and > why the fixes to the various edge case bug fixes have invariably created > new edge cases. It is probably easier to explain by considering an > example. Using the echo tag from the newly added EL test

Re: EL issues and 6.0.x release

2010-01-08 Thread Mark Thomas
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 rule

Re: EL issues and 6.0.x release

2010-01-08 Thread Mark Thomas
On 08/01/2010 11:53, Remy Maucherat wrote: > On Fri, 2010-01-08 at 11:36 +, Mark Thomas wrote: >> I have some rough code that implements this scheme. I am in the process >> of integrating it into Jasper for trunk. I can see this taking little >> while to iron out the wrinkles before I'll be rea

Re: EL issues and 6.0.x release

2010-01-08 Thread Tim Funk
+1 to trunk -0 for backport to 6.0.x Is this legal? ${'\'}${1+1} Shouldn't it be: ${'\\'}${1+1} Or phrased another way, what should I see on the screen with this: ${1+1}${'\t'}${1+1} "2 2" or "2\t2" -Tim On 1/8/2010 6:36 AM, Mark Thomas wrote: One area where help would be appreciated is in

Re: EL issues and 6.0.x release

2010-01-08 Thread Remy Maucherat
On Fri, 2010-01-08 at 11:36 +, Mark Thomas wrote: > I have some rough code that implements this scheme. I am in the process > of integrating it into Jasper for trunk. I can see this taking little > while to iron out the wrinkles before I'll be ready t propose a > back-port for 6.0.x. I don't th