Julian Foad <julianfoad_at_btopenworld.com> writes:
> Hi Philip. Amidst the rest of the discussion perhaps you missed this:
>
> Julian Foad wrote:
>> Index: subversion/include/svn_fs.h
>> ===================================================================
>> Â * For each path in @a targets @a lock_callback will be invoked
>> Â * passing @a lock_baton and the lock and error that apply to path.
>> Â * @a lock_callback can be NULL in which case it is not called.
>> + *Â Â ### Implementation here requires lock_callback is non-null.
>
> That's on svn_fs_lock_many(). The implementation contains:
>
> Â for (hi = ...)
> Â Â Â Â Â ...
> Â Â Â Â Â if (err)
> Â Â Â Â Â Â Â {
> Â Â Â Â Â Â Â Â Â if (!cb_err)
> Â Â Â Â Â Â Â Â Â Â Â cb_err = lock_callback(lock_baton, svn__apr_hash_index_key(hi),
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â NULL, err, scratch_pool);
> Â Â Â Â Â Â Â Â Â svn_error_clear(err);
> Â Â Â Â Â Â Â }
>
> where lock_callback is called even if it is null, as far as I can tell.
I'd fix it but my password reset hasn't worked yet :-(
--
Philip Martin | Subversion Committer
WANdisco // *Non-Stop Data*
Received on 2014-04-14 20:13:40 CEST