commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Henri Biestro (JIRA)" <j...@apache.org>
Subject [jira] Commented: (JEXL-35) Final API requirements
Date Mon, 20 Jul 2009 10:07:14 GMT

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


Mime
View raw message