[ http://issues.apache.org/jira/browse/JEXL-17?page=all ]
Rahul Akolkar updated JEXL-17: ------------------------------ Fix Version/s: 1.1 Affects Version/s: 1.0 (was: Nightly Builds) > ExpressionFactory.createNewExpression should throw an Exception in case of a > parsing error, not Error > ----------------------------------------------------------------------------------------------------- > > Key: JEXL-17 > URL: http://issues.apache.org/jira/browse/JEXL-17 > Project: Commons JEXL > Issue Type: Improvement > Affects Versions: 1.0 > Reporter: Kohsuke Kawaguchi > Fix For: 1.1 > > Attachments: JEXL-17.diff > > > When ExpressionFactory.createNewExpression(...) takes an syntactically > incorrect expression, JEXL throws TokenMgrError. However, typical callers > (such as Jelly) don't expect such syntax error to be a java.lang.Error, so it > fails to catch it. > A typical outcome is therefore the thread to die completely, and even worse > the stack trace doesn't report neither what was the expression that was being > parsed, nor does it report any contextual information (again in case of > Jelly, that would be where in the jelly script this error happened.) > This makes the error diagnosis unnecessarily hard. I believe the proper thing > to do is to wrap such an exception into java.lang.Exception or its sub-class, > allowing the caller to catch it. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]