On Mon, 05 Mar 2007, Malcolm Rowe wrote:
> On Sun, Mar 04, 2007 at 09:27:09PM -0800, firstname.lastname@example.org wrote:
> > * subversion/libsvn_client/copy.c
> > (extend_wc_merge_info, wc_to_repos_copy): svn_mergeinfo_merge() creates a new
> > apr_hash_t * for its output hash, so don't make *output the same as either
> > of the hashes being merged or the mergeinfo in that hash will be lost.
> Maybe the signature of svn_mergeinfo_merge() should change to only
> accept two hashes, and merge the latter into the former, if that's the
> common case? (you could always decide to clone the destination hash
> first, if you decided you needed to keep a copy around).
This is currently the common case. Though what Malcolm suggests is
not the prettiest incarnation of this API, it does seem like the most
efficient and most useful. +1
Received on Mon Mar 5 07:47:40 2007
- application/pgp-signature attachment: stored