axis-java-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Deepal Jayasinghe <dee...@opensource.lk>
Subject Re: [Axis2] Problems with MessageContext
Date Wed, 12 Jul 2006 16:03:04 GMT
If you are happy with unmodifiable map then I have no objection , any
way if some one want to modify property (or properties) he can easily do
that no need to call the getProperties and then modify the map.

So lets keep what we have now  :)

Carsten Ziegeler wrote:

>Carsten Ziegeler schrieb:
>  
>
>>Deepal Jayasinghe wrote:
>>    
>>
>>>Hi Carsten;
>>>
>>>I went through your patch and realized that , you are giving read only
>>>access to the properties . So if someone want to change the value of
>>>property (properties) then he wont be able to do that. I think we need
>>>to provide way to change them as well .
>>>
>>>      
>>>
>>Hmm, but in this case has the MessageContext no control about what has
>>been changed, because it happens outside of its context.
>>
>>But I will come up with another patch which will handle put and remove
>>as well.
>>
>>    
>>
>Ok, I look at a possible solution - but I think it's not worth the effort :)
>In order to be able to change the map, it's requrired to provide a map
>implementation which acts as a wrapepr around all the other maps. Simple
>operations like put,get,remove etc. are simple to implement - but
>complex ones, like values(), entrySet() etc. require a lot of code.
>We could use commons-collections and their CompositeMap for that - at
>least partically.
>
>Before I implement this, I would like to know if adding a dependency to
>commons collections just for this simple feature is fine. Personally I
>think the getProperties() method should return an unmodifiable map.
>
>Carsten
>
>
>  
>

-- 
Thanks,
Deepal
................................................................
~Future is Open~ 




---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org


Mime
View raw message