commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dIon Gillard <d...@multitask.com.au>
Subject Re: [httpclient] patches for HttpClient.java, HttpConnection.java
Date Sun, 10 Feb 2002 14:29:50 GMT
Sean C. Sullivan wrote:

> 
> I updated the code for the HttpClient:
> 
>         HttpClient.java   
>         HttpConnection.java
>
>Patch files attached.
> 
> I created the patch files using this command:
>  
>              cvs diff -c Foo.java > Foo.patch
>  
>-Sean
>
Hi Sean,

for the HttpConnection patch, you've used a throw new 
NullPointerException. From an API point of view, this makes more sense 
(IMHO) as a IllegalArgumentException. What do you think?

-- 
dIon Gillard, Multitask Consulting
http://www.multitask.com.au/developers





--
To unsubscribe, e-mail:   <mailto:commons-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:commons-dev-help@jakarta.apache.org>


Mime
View raw message