commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rahul Akolkar (JIRA)" <>
Subject [jira] Updated: (JEXL-58) UnifiedJEXL
Date Wed, 15 Jul 2009 03:55:14 GMT


Rahul Akolkar updated JEXL-58:

    Assignee: Rahul Akolkar

I've committed the patch to the 2.0 branch in r794133, it'd be great if you could double check:

Thanks for the contribution.

Some follow-up comments (so leaving this issue unresolved):
 * I like that the 1.0 line uses a checked exception from Expression#evaluate(...) -- this
ensures that users actually think about how JEXL clients should cope when expression evaluation
blows up. I see that the change was part of JEXL-55 when JexlException was added (as a RuntimeException).
Fine with using a narrower class here (though we should ask if it is really worth the signature
change to a core API) but I'd prefer to keep it as a checked exception.
 * There are some traces to console when running the tests, best to avoid that IMO (by handling
those exceptions in test code).

> UnifiedJEXL
> -----------
>                 Key: JEXL-58
>                 URL:
>             Project: Commons JEXL
>          Issue Type: New Feature
>            Reporter: Henri Biestro
>            Assignee: Rahul Akolkar
>             Fix For: 2.0
>         Attachments: JEXL-58.patch, JEXL-58.patch, JEXL-58.patch
>  An evaluator similar to the unified EL evaluator used in JSP/JSF based on JEXL.
>  It is intended to be used in configuration modules, XML based frameworks or JSP taglibs.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message