commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Simone Tripodi (Commented) (JIRA)" <>
Subject [jira] [Commented] (CHAIN-58) Update Chain Context interface to use K,V generics
Date Mon, 05 Dec 2011 09:23:40 GMT


Simone Tripodi commented on CHAIN-58:

Hi Elijah!

I am prototyping on my local repo, after applied your patch, something like:

interface Command<K, V, C extends Context<K, V>>

In this way, commands already know wich kind of keys/values and context they manipulate -
of course it implies a lot of modifications in the current code.

If it sounds reasonable to you and you have cycles to provide a patch, it would be wonderful
- I don't have a lot of time to develop it myself during these months, but I can review patches.
Moreover I would like you continue leadering that idea ;)

Last recommendation: please turn off tabs in your IDE and use spaces instead :P

Many thanks in advance for your hard work!
> 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.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