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