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

Re: [PATCH] issue #3555: make svn_stream_readline() a stream method

From: Stefan Sperling <stsp_at_elego.de>
Date: Thu, 8 Jul 2010 15:23:24 +0200

On Thu, Jul 08, 2010 at 02:08:15PM +0200, Stefan Sperling wrote:
> On Thu, Jul 08, 2010 at 12:39:22PM +0100, Julian Foad wrote:
> > I've only skimmed through it so far, and probably can't do a full review
> > before the end of the month. However from what I've seen it looks good
> > enough to go ahead and then improve/fix later.
> OK, I'll fix up the bits above and commit.

FYI, on IRC, we decided to go for a different approach:

<stsp> julianf, look at svn_hunk_t in svn_diff.h
<stsp> the streams in there need special handling when reading lines from them
<stsp> instead of overloading readline we can simply add a set of helper functions that are meant to be used for reading lines from those streams
<stsp> that is specific to what we need in svn patch,
<stsp> and avoids having to worry about all the other generic streams
<stsp> and if a user calls the diff-specific stream reader functions on a generic stream, that's the user's problem
<julianf> Yes, that sounds better.
<stsp> ok, I'll do that then
Received on 2010-07-08 15:24:06 CEST

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