[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1130118 - in /subversion/trunk/subversion: include/svn_io.h libsvn_subr/stream.c libsvn_subr/subst.c

From: Stefan Sperling <stsp_at_elego.de>
Date: Thu, 2 Jun 2011 22:59:15 +0200

On Thu, Jun 02, 2011 at 04:05:49PM +0200, Stefan Sperling wrote:
> On Thu, Jun 02, 2011 at 05:40:41PM +0400, Ivan Zhakov wrote:
> > On Thu, Jun 2, 2011 at 17:36, Greg Stein <gstein_at_gmail.com> wrote:
> > > Well... note that the stream concept is used rather than apr_file_t
> > > because "we may want to use something besides a file, in the future".
> > > IOW, these extra APIs are for unproven future need.
> > >
> > > I might also argue for something like:
> > >
> > > svn_seekable_create()
> > >
> > > And the associated 'svn_seekable_t' has the various forms of seeking.
> > > It could consume a string, or a file, and it could also use the same
> > > kind of read/write callback types. But it would have functions that
> > > svn_stream_t does not have. A "stream" is generally not seekable,
> > > under any interpretation of the word.
> > >
> > +1.
>
> Then we need to move the functionality out of the stream code and
> into the patch code. Which is fine with me but this would be a release
> blocker since it involves changes to the public API and I don't know
> how fast it can be done. The patch code uses this quite heavily.

I've ripped out mark/seek from the patch code in r1130809. I think we
could now remove mark and seek from the stream API before 1.7 release.
Received on 2011-06-02 22:59:57 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.