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

Re: svn commit: r1626023 - in /subversion/branches/move-tracking-2/subversion: include/private/svn_editor3.h libsvn_delta/compat3.c libsvn_ra/ra_loader.c libsvn_repos/commit.c libsvn_wc/util.c libsvn_wc/wc.h

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Fri, 26 Sep 2014 13:07:32 +0100

Branko Čibej wrote:
>> URL: http://svn.apache.org/r1626023
>> Log:
>> On the 'move-tracking-2' branch: Simplify editor shim usage by passing file
>> content in memory instead of in temporary files.
>>
>> The shim code was leaving temporary files in /tmp. Managing temporary files
>> correctly (for example, cleaning up even in case of error) is difficult, and
>> handling of text content too big to fit in memory is unnecessary at this
>> stage of development.
>
> This is why spillbufs exist, and why the serf push/pull adapter creates
> tempfiles in .svn/tmp. Why don't you just use the same mechanism?

That occurred to me. For my purposes here the priority is ease of prototyping, and I'm more familiar with stringbuf semantics than with spillbuf. When we need release-quality code then presumably spillbuf is the way to go, I agree.

- Julian
Received on 2014-09-26 14:14:47 CEST

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