Am 16.06.2013 um 13:46 schrieb Benedikt Ritter <benerit...@gmail.com>:

> 
> Am 15.06.2013 um 15:52 schrieb Benedikt Ritter <benerit...@gmail.com>:
> 
>> 
>> Am 14.06.2013 um 22:36 schrieb Benedikt Ritter <benerit...@gmail.com>:
>> 
>>> Hi,
>>> 
>>> the o.a.c.chain2.generic.CopyCommand IMHO really implements two commands:
>>> 
>>> * a CopyCommand that copies a value from one key to another
>>> * an OverrideCommand, that overrides a value with a provided value.
>> 
>> We could even extract a RemoveCommand from CopyCommand. A RemoveCommand 
>> simply removes getKey() from the context.
> 
> I have created CHAIN-91 [1] for this. If no one objects, I'll implement this 
> tonight.

Just realized we already have a RemoveCommand… So no need to reimplement it. I 
think is makes sense to remove the "remove"-functionality from CopyCommand 
then...

> 
> Benedikt
> 
> [1] https://issues.apache.org/jira/browse/CHAIN-91
> 
>> 
>>> 
>>> What do you think about splitting this up into two separate classes?
>>> 
>>> Regards,
>>> Benedikt
>> 
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to