[
https://issues.apache.org/jira/browse/JEXL-35?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12733179#action_12733179
]
Henri Biestro commented on JEXL-35:
-----------------------------------
I think the Ubespect also needs to be reworked; it should be the class to
derive to set/get properties & execute methods. Besides, the original Velociy
ties dont really bring value here and some of the code in UberspectImpl is not
really as clear as I'd personally like. And there is some broken simmetry
between get & set ("gettable" vs map) in method discovery.
As of packaging goes, it seems a jexl.introspection package should be the place
to put all these in; however, it might not be possible if we are to maintain
compatiblity at this level.
I'll submit a patch soon without the re-packaging to have a better view.
> 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.
