Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
Message-ID: <100073199.1247586675149.JavaMail.jira@brutus>
Date: Tue, 14 Jul 2009 08:51:15 -0700 (PDT)
From: "Rahul Akolkar (JIRA)" <jira@apache.org>
To: issues@commons.apache.org
Subject: [jira] Commented: (JEXL-58) UnifiedEL
In-Reply-To: <1502369221.1247520795591.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ 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.