commons-issues mailing list archives

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

    [ https://issues.apache.org/jira/browse/CHAIN-58?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13162683#comment-13162683
] 

Simone Tripodi commented on CHAIN-58:
-------------------------------------

Hi Elijah!

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

{code}
interface Command<K, V, C extends Context<K, V>>
{code}

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!
-Simo
                
> 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.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