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:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to