You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by Ortwin Glück <or...@nose.ch> on 2002/09/05 10:43:47 UTC
[HttpClient][PATCH] Fix deprecation warnings
Patch fixes deprecation warnings in the tests introduced by the
architecture change of the status line.
Could someone apply this for me please. I have no access to the
repository yet. Sorry it's huge (43KB) :-)
Patch is for the src/ directory.
Re: [HttpClient][PATCH] Fix deprecation warnings
Posted by Jeff Dever <js...@sympatico.ca>.
I personally favour thin interfaces, which is why I marked them as deprecated
(don't need two ways to get the same information). But HttpMethod isn't exactly
"thin" anyway. Either way is fine with me if somone feels strongly about it.
Ortwin Glück wrote:
> Jeff Dever wrote:
> > Just to confirm before I apply this patch: is it right that those methods be
> > deprecated? There were no comments to the contrary when I posted the patch
> > for review earlier.
>
> When I start thinking about it I find the convenience method quite handy
> actually. Maybe we should just leave it? No idea what's the better solution.
>
> --
> To unsubscribe, e-mail: <ma...@jakarta.apache.org>
> For additional commands, e-mail: <ma...@jakarta.apache.org>
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>
Re: [HttpClient][PATCH] Fix deprecation warnings
Posted by Ortwin Glück <or...@nose.ch>.
Jeff Dever wrote:
> Just to confirm before I apply this patch: is it right that those methods be
> deprecated? There were no comments to the contrary when I posted the patch
> for review earlier.
When I start thinking about it I find the convenience method quite handy
actually. Maybe we should just leave it? No idea what's the better solution.
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>
Re: [HttpClient][PATCH] Fix deprecation warnings
Posted by Jeff Dever <js...@sympatico.ca>.
Just to confirm before I apply this patch: is it right that those methods be
deprecated? There were no comments to the contrary when I posted the patch
for review earlier.
Ortwin Glück wrote:
> Patch fixes deprecation warnings in the tests introduced by the
> architecture change of the status line.
>
> Could someone apply this for me please. I have no access to the
> repository yet. Sorry it's huge (43KB) :-)
>
> Patch is for the src/ directory.
>
> ------------------------------------------------------------------------
> Name: patch.diff
> patch.diff Type: Plain Text (text/plain)
> Encoding: 7bit
>
> Part 1.3Type: Plain Text (text/plain)
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>