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

Re: svn commit: r1072519 - Add svn_stream_skip(), svn_stream_buffered(), svn_stream_supports_mark()

From: Julian Foad <julian.foad_at_wandisco.com>
Date: Thu, 02 Jun 2011 14:31:23 +0100

On Thu, 2011-06-02 at 14:32 +0300, Daniel Shahaf wrote:
> Stefan Fuhrmann wrote on Thu, Jun 02, 2011 at 00:08:16 +0200:
> > On 01.06.2011 16:08, Julian Foad wrote:
> > >On Sun, 2011-02-20, stefan2_at_apache.org wrote:
> > >>Merge all changes (r1068695 - r1072516) from the
> > >>integrate-stream-api-extensions.
> > >>
> > >>These patches add svn_stream_skip(), svn_stream_buffered()
> > >>and svn_stream_supports_mark().
> > >A belated review.
> > >
> > <snip>
> ...
> > >The relevant question right now is: is there a clear benefit for this
> > >being public in 1.7? I think not, so I propose to make the "is
> > >buffered" API private by renaming the relevant symbols to
> > >"svn_stream__*". Or move them into private/svn_io_private.h, as Stefan
> > >just suggested. Questions of semantics can then wait.
> > Make it private. It simply felt a little odd to require
> > every stream to implement a private API as well.
> >
>
> Agreed, since the stream API is public it needs to work for streams
> defined by third parties too. I suppose we could implement this by
> answering "FALSE" by default in _is_buffered() and _supports_mark().

Made private in 1130538.

- Julian
Received on 2011-06-02 15:32:00 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.