RE: svn commit: r1555716 -/subversion/trunk/subversion/libsvn_ra_serf/update.c

2014-01-09 Thread Bert Huijben
> -Original Message- > From: Ivan Zhakov [mailto:i...@visualsvn.com] > Sent: donderdag 9 januari 2014 10:18 > To: Bert Huijben > Cc: Subversion Development > Subject: Re: svn commit: r1555716 - > /subversion/trunk/subversion/libsvn_ra_serf/update.c > > On

Re: svn commit: r1555716 -/subversion/trunk/subversion/libsvn_ra_serf/update.c

2014-01-09 Thread Branko Čibej
On 09.01.2014 10:38, Ivan Zhakov wrote: > On 9 January 2014 13:29, Branko Čibej wrote: >> On 09.01.2014 10:18, Ivan Zhakov wrote: >> >> On 7 January 2014 18:29, Bert Huijben wrote: >> >> On 7 January 2014 12:42, Bert Huijben wrote: >> >> The current code works in both scenarios (tested via buffe

Re: svn commit: r1555716 -/subversion/trunk/subversion/libsvn_ra_serf/update.c

2014-01-09 Thread Ivan Zhakov
On 9 January 2014 13:29, Branko Čibej wrote: > On 09.01.2014 10:18, Ivan Zhakov wrote: > > On 7 January 2014 18:29, Bert Huijben wrote: > > On 7 January 2014 12:42, Bert Huijben wrote: > > The current code works in both scenarios (tested via buffersizes of just a > few bytes, and a debugger veri

Re: svn commit: r1555716 -/subversion/trunk/subversion/libsvn_ra_serf/update.c

2014-01-09 Thread Branko Čibej
On 09.01.2014 10:18, Ivan Zhakov wrote: > On 7 January 2014 18:29, Bert Huijben wrote: >> On 7 January 2014 12:42, Bert Huijben wrote: >>> The current code works in both scenarios (tested via buffersizes of just a >>> few bytes, and a debugger verifyimg the corner cases, etc). But I doubt it >>>

Re: svn commit: r1555716 -/subversion/trunk/subversion/libsvn_ra_serf/update.c

2014-01-09 Thread Ivan Zhakov
On 7 January 2014 18:29, Bert Huijben wrote: > On 7 January 2014 12:42, Bert Huijben wrote: >> The current code works in both scenarios (tested via buffersizes of just a >> few bytes, and a debugger verifyimg the corner cases, etc). But I doubt it >> is worth all the trouble. The file is null unt

RE: svn commit: r1555716 -/subversion/trunk/subversion/libsvn_ra_serf/update.c

2014-01-07 Thread Bert Huijben
ment" Subject: Re: svn commit: r1555716 -/subversion/trunk/subversion/libsvn_ra_serf/update.c On 7 January 2014 12:42, Bert Huijben wrote: > The current code works in both scenarios (tested via buffersizes of just a > few bytes, and a debugger verifyimg the corner cases, etc). But I doub

Re: svn commit: r1555716 - /subversion/trunk/subversion/libsvn_ra_serf/update.c

2014-01-07 Thread Ivan Zhakov
On 7 January 2014 12:42, Bert Huijben wrote: > The current code works in both scenarios (tested via buffersizes of just a > few bytes, and a debugger verifyimg the corner cases, etc). But I doubt it > is worth all the trouble. The file is null until something is spilled and > from that point onwar

Re: svn commit: r1555716 - /subversion/trunk/subversion/libsvn_ra_serf/update.c

2014-01-07 Thread Bert Huijben
The current code works in both scenarios (tested via buffersizes of just a few bytes, and a debugger verifyimg the corner cases, etc). But I doubt it is worth all the trouble. The file is null until something is spilled and from that point onwards everything is in the file (until the spill is em

Re: svn commit: r1555716 - /subversion/trunk/subversion/libsvn_ra_serf/update.c

2014-01-06 Thread Ivan Zhakov
On 6 January 2014 15:23, wrote: > Author: rhuijben > Date: Mon Jan 6 11:23:05 2014 > New Revision: 1555716 > > URL: http://svn.apache.org/r1555716 > Log: > In the ra serf (update) editor, spool the report to a spillbuffer instead of > to a tempfile to avoid creating a tempfile for small requests