commons-issues mailing list archives

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


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:

[ERROR] /Users/simonetripodi/Documents/workspace/commons-chain/src/main/java/org/apache/commons/chain/config/[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

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:
>             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,
> As discussed in the mailing list, I am suggesting that we change the definition of Context
> {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:!default.jspa
For more information on JIRA, see:


View raw message