commons-issues mailing list archives

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

     [ https://issues.apache.org/jira/browse/JEXL-58?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

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:

  http://svn.apache.org/viewvc?view=rev&revision=794133

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: https://issues.apache.org/jira/browse/JEXL-58
>             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.


Mime
View raw message