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

Henri Biestro commented on JEXL-35:
-----------------------------------

Package structure looks good & clear;
It seems extensibility & configurability are covered & trying to reduce 
visibility of some classes would reduce what we gained on these aspects.
The last "may be" is whether we should keep/get rid of Arithmetic and make 
JexlArithmetic public (adding a toString() to it for completeness); and other 
methods as protected; there does not seem to be much value to Arithmetic as an 
interface by itself imho.


> Final API requirements
> ----------------------
>
>                 Key: JEXL-35
>                 URL: https://issues.apache.org/jira/browse/JEXL-35
>             Project: Commons JEXL
>          Issue Type: Improvement
>    Affects Versions: 2.0
>            Reporter: dion gillard
>             Fix For: 2.0
>
>
> Which classes should be public?
> Is the package structure reasonable? 
> Are classes in the right packages?
> Is it easily extensible?
> ...

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

Reply via email to