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] Commented: (JEXL-58) UnifiedEL
Date Tue, 14 Jul 2009 15:51:15 GMT

    [ https://issues.apache.org/jira/browse/JEXL-58?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12730926#action_12730926
] 

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 (Asserter.java, ScriptFactory.java)
 * I'd avoid changing imports style where not strictly necessary (Debugger.java)
 * Some changes WRT throws clauses in signatures (whether thats strictly necessary)
 * In spirit with surrounding code, braces for ifs


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


Mime
View raw message