avalon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From J Aaron Farr <fa...@apache.org>
Subject Re: [VOTE] DefaultConfiguration.setAttribute(String,<null>)
Date Wed, 10 Dec 2003 10:21:22 GMT
On Wed, 2003-12-10 at 14:35, 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.
> 
> /LS

     I. +1 for 4 as well.
-- 
 jaaron  <http://jadetower.org>


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


Mime
View raw message