--- On Fri, 2/19/10, Adam Heath <doo...@brainfood.com> wrote:
> What is getOriginal supposed to
> return?  The original string, or one
> that has been fixed up and normalized?

It is supposed to return the original string.

> What is toString supposed to return?  The same as
> getOriginal?

The same as getOriginal. I added that override recently as a convenience for 
debugging.

> if there was a parse error, a
> missing trailing
> '}', should toString add this missing character?

Wouldn't that be confusing or make it harder to debug bad expressions?

> Basically, what is supposed to happen when parsing a badly
> formatted
> string, ie, "The class for [${a}] is ${a.getClass()".

Throw an exception. In other words, force the programmer to fix it.

> Additionally, ${bsh:} and ${groovy:} weren't returning the
> 'bsh' or
> 'groovy' part in original.

Oops.

> I don't have coverage on any other helper
> classes in that
> folder, including those that are called by FSE.

I could help out with that. I might skip the UEL functions though - since  
there are so many of them and most of them call java.* classes.

-Adrian




Reply via email to