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

I've recorded this point in the BRANCH-README todo list.

Greg Stein wrote on Thu, Jul 14, 2011 at 04:42:54 -0400:
> 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 14:25:57 CEST