avalon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Leo Simons <leosim...@apache.org>
Subject Re: [VOTE] DefaultConfiguration.setAttribute(String,<null>)
Date Wed, 10 Dec 2003 19:00:29 GMT
Leo Sutic wrote:
> When DefaultConfiguration.setAttribute(key,value) is called with a null 
> as the value parameter, it should...
> 
>  1. ...throw a NullPointerException.
> 
>  2. ...throw an IllegalArgumentException.
> 
>  3. ...act as if called with DefaultConfiguration.setAttribute( key, ""
> ). 
>        I.e. set the attribute to the empty string.
> 
>  4. ...remove the attribute if it exists.
> 
> My vote is for 4. Why? Because setValue(null) removes any existing
> value from a DefaultConfiguration.

+1. That's what the collection APIs do, innit?

- LSD



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


Mime
View raw message