Re: svn commit: r1144478 - in /subversion/branches/revprop-packing/subversion: include/svn_io.h libsvn_subr/stream.c tests/libsvn_subr/stream-test.c

On Jul 14, 2011 4:35 AM, "Julian Foad" <julian.foad_at_wandisco.com> wrote:
>
> On Fri, 2011-07-08, hwright_at_apache.org wrote:
> > New Revision: 1144478
> >
> > URL: http://svn.apache.org/viewvc?rev=1144478&view=rev
> > Log:
> > On the revprop-packing branch:
> > Rev the svn_stream_copy3() API and give it an argument which allows the
caller
> > to specify the number of bytes to copy.
>
> I notice we have about 24 existing callers of svn_stream_copy3(), and I
> don't suppose any of them want to specify the number of bytes to copy,
> so I think this might be better as a separate API with a new name rather
> than a revision of the existing API.

That's what was bugging me. You put your finger on it.

Fully agreed.

Cheers,
-g
Received on 2011-07-14 10:43:36 CEST