Julian Foad <julianf...@btopenworld.com> writes: > Hi Philip... > >> URL: http://svn.apache.org/r1577280 >> Log: >> Merge the fsfs-lock-many branch to trunk. [...] > >> * 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 >> (svn_repos_fs_lock2, svn_repos_fs_unlock2): new. > > Do we intend to deprecate the old versions? If not, then it would be > better to name the new functions something like ...lock_many() instead > of ...lock2().
I'm unsure. In some ways it is convenient to have a single path function as it avoids the need to construct hashes and the error handling is simpler and less prone to leak. On the other hand when svn_client_copy4 introduced multi-path copy source the single path version was deprecated. -- Philip Martin | Subversion Committer WANdisco // *Non-Stop Data*
