geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From James Strachan <james_strac...@yahoo.co.uk>
Subject Re: [PATCH] jsr88 DeploymentFactoryManager implementation + unittests
Date Thu, 14 Aug 2003 06:57:40 GMT

On Thursday, August 14, 2003, at 06:31  am, Aaron Mulder wrote:

> Maas,
>
> 	Regarding this particular patch, I'd prefer that we don't actually
> add "null" to the list of DeploymentFactories, even if it's passed as 
> an
> argument to DeploymentFactoryManager.registerDeploymentFactory().  
> That's
> silly, we should just ignore a null.

Failing fast with silly arguments is a good idea - e.g. throw an 
IllegalArgumentException or NullPointerException.

James
-------
http://radio.weblogs.com/0112098/


Mime
View raw message