Re: svn commit: r1582845 - /subversion/trunk/subversion/libsvn_fs/fs-loader.c

2014-03-28 Thread Philip Martin
"Bert Huijben" writes: >> + expiration_date, steal_lock, pool, pool); >> + >> + if (apr_hash_count(results)) > > Is this function explicitly documented to always set the results value > on all error paths? > > I don't see that in the documentation for svn_fs_lock2? It is now

Re: Review of lock-many API

2014-03-28 Thread Philip Martin
Julian Foad writes: > Philip Martin wrote: >> Julian Foad writes:   URL: http://svn.apache.org/r1577280   * subversion/include/svn_fs.h     (svn_fs_lock_target_t, svn_fs_lock_result_t,      svn_fs_lock2, svn_fs_unlock2): new.   * subversion/include/svn_repos.h     (s

RE: svn commit: r1582845 - /subversion/trunk/subversion/libsvn_fs/fs-loader.c

2014-03-28 Thread Bert Huijben
> -Original Message- > From: phi...@apache.org [mailto:phi...@apache.org] > Sent: vrijdag 28 maart 2014 18:51 > To: comm...@subversion.apache.org > Subject: svn commit: r1582845 - /subversion/trunk/subversion/libsvn_fs/fs- > loader.c > > Author: philip > Date: Fri Mar 28 17:51:09 2014 >

parseability fixes to STATUS (was: svn commit: r1582552 - /subversion/branches/1.8.x/STATUS)

2014-03-28 Thread Daniel Shahaf
rhuij...@apache.org wrote on Thu, Mar 27, 2014 at 23:52:27 -: > Author: rhuijben > Date: Thu Mar 27 23:52:27 2014 > New Revision: 1582552 > > URL: http://svn.apache.org/r1582552 > Log: > * STATUS: Fix minor layout issues to avoid problems with backport script. > > Modified: > subversion/b