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