[ 
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

        

Reply via email to