You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Edmund Wong <ed...@kdtc.net> on 2009/04/07 14:43:32 UTC

Re: svn commit: r37069 - trunk/subversion/libsvn_subr

Hyrum K. Wright wrote:
> Author: hwright
> Date: Tue Apr  7 07:05:26 2009
> New Revision: 37069
> 
> Log:
> Shuffle a couple of deprecated functions.
> 
> * subversion/libsvn_subr/stream.c
>   (svn_stream_copy2, svn_stream_copy3): Move from here...
> 
> * subversion/libsvn_subr/deprecated.c
>   (svn_stream_copy2, svn_stream_copy3): ...to here.

Pardon my ignorance, but shouldn't that be just
svn_stream_copy and not svn_stream_copy3?

The reason why I'm asking is because in the
svn_io.h that I had printed out, it shows
that svn_stream_copy3 was introduced in 1.6,
so it's strange to deprecate it now.  That
and because I was *just* talking about this
function in dev and that there was supposed
to be a bite-sized 'patch-to-be' with this
function and I was thinking of ways to tackle
this bite-sized task.


Thanks for the clarification.

Edmund

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1578164

Re: svn commit: r37069 - trunk/subversion/libsvn_subr

Posted by "Hyrum K. Wright" <hy...@mail.utexas.edu>.
On Apr 7, 2009, at 9:47 AM, Edmund Wong wrote:

> Never mind.  I just saw Greg Stein's changes.
> Sorry for the fuss.

No fuss at all.  Thanks for the review.

-Hyrum

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1578233

Re: svn commit: r37069 - trunk/subversion/libsvn_subr

Posted by Edmund Wong <ed...@kdtc.net>.
Never mind.  I just saw Greg Stein's changes.
Sorry for the fuss.

Edmund

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1578174