Re: Does svn_fs_change_merge_info() accept NULL as mergeinhash

On Thu, 05 Apr 2007, Kouhei Sutou wrote:

> Hi,
>
> svn_fs_change_merge_info()'s document says:
>
> - @a mergeinhash is the new value of the mergeinfo for PATH, or NULL if
> the merge info for that path should be removed altogether.
>
> But the implementation of fsfs,
> subversion/libsvn_fs_fs/tree.c:fs_change_merge_info(), doen't accept NULL.
> Which is correct?

We need to be able to handle the case where a path has all its merge
info removed. I think this should be represented by a hash with paths
(keys) mapping to empty rangelists (apr_array_header_t *'s containing
zero svn_merge_range_t *'s).

We also need to handle the case where all merge info for a path has
been removed.

I think that svn_fs_change_merge_info() should accept NULL to handle
this latter case, since apr_hash_t *'s don't handle NULL values
(setting a NULL value equates to removing the mapping entirely).