RE: [HttpClient] [PATCH] HttpMultiClient Catches IOException Twic e

["Elwin, Martin" <me...@rsasecurity.com>]

> From: Adrian Sutton [mailto:adrian.sutton@ephox.com] 
> In HttpMultiClient (latest version in CVS) there is a catch 
> statement for
> IOException followed by HttpException, however since 
> HttpException extends
> from IOException there is no possible way the HttpException 
> catch clause can
> be activated.

Seems like a problem introduced when changing the super class of
HttpException from Exception to URIException (which inherits IOException).
>From the source of HttpException:

"Why is it from URIException?
 To simplify the programming style for the inherited exception instances."

Not sure if I'm totally convinced, though.

Anyone care to motivate why this is a Good Thing?

Kindly,

/Martin



--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>

Re: [HttpClient] [PATCH] HttpMultiClient Catches IOException Twice

[Sung-Gu <je...@apache.org>]

----- Original Message -----
From: "Elwin, Martin" <me...@rsasecurity.com>


> > From: Adrian Sutton [mailto:adrian.sutton@ephox.com]
>
> Seems like a problem introduced when changing the super class of
>
> "Why is it from URIException?
>  To simplify the programming style for the inherited exception instances."
>
> Anyone care to motivate why this is a Good Thing?

His patch show us that the 4 lines turn to 2 lines, isn't it?  ;)

Sung-Gu


--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>