commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rahul Akolkar (JIRA)" <>
Subject [jira] Commented: (JEXL-58) UnifiedEL
Date Tue, 14 Jul 2009 15:51:15 GMT


Rahul Akolkar commented on JEXL-58:

Pasting from the dev list thread, lets continue here:

This is useful. I'd like to suggest we call the class UnifiedJEXL (ofcourse, there is no "unification"
in JEXL, but I think it will be unambiguous and still convey the right meaning at the same
time -- atleast it does so for me).

At a quick glance, I only have some minor nits with the patch:
 * Some files have whitespace only changes (,
 * I'd avoid changing imports style where not strictly necessary (
 * Some changes WRT throws clauses in signatures (whether thats strictly necessary)
 * In spirit with surrounding code, braces for ifs

> UnifiedEL
> ---------
>                 Key: JEXL-58
>                 URL:
>             Project: Commons JEXL
>          Issue Type: New Feature
>            Reporter: Henri Biestro
>             Fix For: 2.0
>         Attachments: 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