Re: [jexl] size method unit test failing

2003-09-09 Thread Tim O'Brien
The offending line in JexlTest has been removed. If you want the size of an array, call "size()". Now, on to the next target. http://jakarta.apache.org/commons/jexl/junit-report.html#JexlTest testNotConditionsWithDots is failing. Specifically, the following code fails: Expression e = Expre

Re: [jexl] size method unit test failing

2003-09-09 Thread Mark H. Wilkinson
On Tue, 2003-09-09 at 13:51, Geir Magnusson Jr. wrote: > > I think that just using size() is the right way to go. The only reason I can see to support foo.length for arrays is if the JSP expression language supports it, on the basis that Jexl claims to be an extension of the JSP expression langua

Re: [jexl] size method unit test failing

2003-09-09 Thread Geir Magnusson Jr .
On Tuesday, September 9, 2003, at 10:00 AM, Tim O'Brien wrote: On Tue, 2003-09-09 at 07:51, Geir Magnusson Jr. wrote: On Tuesday, September 9, 2003, at 08:25 AM, Tim O'Brien wrote: On Tue, 2003-09-09 at 06:58, Geir Magnusson Jr. wrote: I guess we were going to figure out if we want to add the art

Re: [jexl] size method unit test failing

2003-09-09 Thread Tim O'Brien
On Tue, 2003-09-09 at 07:51, Geir Magnusson Jr. wrote: > On Tuesday, September 9, 2003, at 08:25 AM, Tim O'Brien wrote: > > > On Tue, 2003-09-09 at 06:58, Geir Magnusson Jr. wrote: > >> I guess we were going to figure out if we want to add the artificial > >> notion of the length field, or just as

Re: [jexl] size method unit test failing

2003-09-09 Thread Geir Magnusson Jr .
On Tuesday, September 9, 2003, at 08:25 AM, Tim O'Brien wrote: On Tue, 2003-09-09 at 06:58, Geir Magnusson Jr. wrote: I guess we were going to figure out if we want to add the artificial notion of the length field, or just ask people to use size(). 'length' is really weird, as it doesn't really

Re: [jexl] size method unit test failing

2003-09-09 Thread Tim O'Brien
On Tue, 2003-09-09 at 06:58, Geir Magnusson Jr. wrote: > I guess we were going to figure out if we want to add the artificial > notion of the length field, or just ask people to use size(). 'length' > is really weird, as it doesn't really exist as a field, and only > applies to arrays. > >

Re: [jexl] size method unit test failing

2003-09-09 Thread Geir Magnusson Jr .
I guess we were going to figure out if we want to add the artificial notion of the length field, or just ask people to use size(). 'length' is really weird, as it doesn't really exist as a field, and only applies to arrays. Why confuse the syntax with an additional way to get size? geir O

Re: [jexl] size method unit test failing

2003-09-09 Thread Mark H. Wilkinson
On Sun, 2003-09-07 at 03:28, Tim O'Brien wrote: > JexlTest line 443: "assertExpression(jc, "array.length", new Integer(5));" > > This is failing because (from what I'm seeing), length isn't given any special > treatment in Parser.jjt. > > Any ideas? I proposed the attached fix a few months bac

[jexl] size method unit test failing

2003-09-06 Thread Tim O'Brien
JexlTest line 443: "assertExpression(jc, "array.length", new Integer(5));" This is failing because (from what I'm seeing), length isn't given any special treatment in Parser.jjt. Any ideas? Tim - To unsubscribe, e-mail: [EMA