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 Fri, 10 Feb 2012 22:06:59 GMT


Simone Tripodi updated CHAIN-58:

    Attachment: CHAIN-58-working-context-generics.patch

The latest attached patch implements {{Chains}} taking in consideration any {{Map<K, V>}}
extension as context (including the existing {{Context}}).

All tests pass.

There are unchecked casts in the XML rules, but, IIRC, XML rules are something we want to
get rid of in 2.0.

There are also binary breakages, sounds it is the right moment to bump to major version.

Please review the patch, if there is an agreement I can commit it and moving [chains] development
> 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-working-context-generics.patch, 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
> {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