commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Simone Tripodi (Updated) (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (CHAIN-58) Update Chain Context interface to use K,V generics
Date Sat, 31 Dec 2011 11:08:30 GMT

     [ https://issues.apache.org/jira/browse/CHAIN-58?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Simone Tripodi updated CHAIN-58:
--------------------------------

    Attachment: chain-58-improved-context-generic.diff

Hi Elijah,

I am still convinced we could do a little step over the basic {{<String, Object>}} implementations,
making base implementations even more generic

Please have a look at the attached {{chain-58-improved-context-generic.diff}} patch, that
operates with type agnostic in generics as mush as possible.

There is anyway a javac issue in the {{config}} package that affects the command line compilation
and not the Eclipse embedded compilation:

{code}
[ERROR] /Users/simonetripodi/Documents/workspace/commons-chain/src/main/java/org/apache/commons/chain/config/ConfigRegisterRule.java:[85,23]
incompatible types; inferred type argument(s) java.lang.Object,java.lang.Object,java.lang.Object
do not conform to bounds of type variable(s) K,V,C
[ERROR] found   : <K,V,C>void
[ERROR] required: void
{code}

Any feedback would be really appreciated, thanks in advance and happy new year!
                
> Update Chain Context interface to use K,V generics
> --------------------------------------------------
>
>                 Key: CHAIN-58
>                 URL: https://issues.apache.org/jira/browse/CHAIN-58
>             Project: Commons Chain
>          Issue Type: Improvement
>    Affects Versions: 2.0
>            Reporter: Elijah Zupancic
>             Fix For: 2.0
>
>         Attachments: chain-58-improved-context-generic.diff, chain-58-with-context-generic.diff,
chain-58.diff
>
>
> As discussed in the mailing list, I am suggesting that we change the definition of Context
from:
> {noformat}
> public interface Context extends Map<String, Object> {
> {noformat}
> to:
> {noformat}
> public interface Context<K, V> extends Map<K, V> {
> {noformat}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message