[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

RE: svn commit: r1149543 - /subversion/trunk/subversion/libsvn_client/mergeinfo.c

From: Bert Huijben <bert_at_qqmail.nl>
Date: Fri, 22 Jul 2011 12:29:33 +0200

> -----Original Message-----
> From: stsp_at_apache.org [mailto:stsp_at_apache.org]
> Sent: vrijdag 22 juli 2011 12:14
> To: commits_at_subversion.apache.org
> Subject: svn commit: r1149543 -
> /subversion/trunk/subversion/libsvn_client/mergeinfo.c
>
> Author: stsp
> Date: Fri Jul 22 10:13:49 2011
> New Revision: 1149543
>
> URL: http://svn.apache.org/viewvc?rev=1149543&view=rev
> Log:
> Convert libsvn_client/mergeinfo.c over to svn_rangelist_merge2().
>
> * subversion/libsvn_client/mergeinfo.c
> (svn_client_mergeinfo_log): Use svn_rangelist_merge2() instead of
> svn_rangelist_merge(). Create and use iterpools as the scratch_pool for
> svn_rangelist_merge2() where appropriate.
>
> Modified:
> subversion/trunk/subversion/libsvn_client/mergeinfo.c
>
> Modified: subversion/trunk/subversion/libsvn_client/mergeinfo.c
> URL:
> http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_client/m
> ergeinfo.c?rev=1149543&r1=1149542&r2=1149543&view=diff
> ==========================================================
> ====================
> --- subversion/trunk/subversion/libsvn_client/mergeinfo.c (original)
> +++ subversion/trunk/subversion/libsvn_client/mergeinfo.c Fri Jul 22
> 10:13:49 2011
> @@ -1847,16 +1847,20 @@ svn_client_mergeinfo_log(svn_boolean_t f
> subtrees. */
> if (apr_hash_count(merged_noninheritable))
> {
> + apr_pool_t *iterpool2 = svn_pool_create(iterpool);
> +
> for (hi = apr_hash_first(iterpool, merged_noninheritable);
> hi;
> hi = apr_hash_next(hi))
> {
> apr_array_header_t *list = svn__apr_hash_index_val(hi);
> - SVN_ERR(svn_rangelist_merge(
> - &master_noninheritable_rangelist,
> + svn_pool_clear(iterpool2);
> + SVN_ERR(svn_rangelist_merge2(
> + master_noninheritable_rangelist,
> svn_rangelist_dup(list, scratch_pool),
> - scratch_pool));
> + scratch_pool, iterpool2));
> }
> + svn_pool_destroy(iterpool2);
> }
>
> /* Find the intersection of the inheritable part of TGT_MERGEINFO
> @@ -1874,6 +1878,7 @@ svn_client_mergeinfo_log(svn_boolean_t f
> to SUBTREE_PATH. */
> apr_array_header_t *subtree_merged_rangelist =
> apr_array_make(scratch_pool, 1, sizeof(svn_merge_range_t *));
> + apr_pool_t *iterpool2 = svn_pool_create(iterpool);
>
> for (hi = apr_hash_first(iterpool, merged);
> hi;
> @@ -1881,15 +1886,17 @@ svn_client_mergeinfo_log(svn_boolean_t f
> {
> apr_array_header_t *list = svn__apr_hash_index_val(hi);
>
> - SVN_ERR(svn_rangelist_merge(&master_inheritable_rangelist,
> - svn_rangelist_dup(list,
> - scratch_pool),
> - scratch_pool));
> - SVN_ERR(svn_rangelist_merge(&subtree_merged_rangelist,
> - svn_rangelist_dup(list,
> + svn_pool_clear(iterpool2);
> + SVN_ERR(svn_rangelist_merge2(master_inheritable_rangelist,
> + svn_rangelist_dup(list,
> + scratch_pool),
> + scratch_pool, iterpool2));
> + SVN_ERR(svn_rangelist_merge2(subtree_merged_rangelist,
> + svn_rangelist_dup(list,
> scratch_pool),
> - scratch_pool));
> + scratch_pool, iterpool2));
> }
> + svn_pool_destroy(iterpool2);
>
> apr_hash_set(inheritable_subtree_merges,
> apr_pstrdup(scratch_pool, subtree_path),
> @@ -1937,9 +1944,9 @@ svn_client_mergeinfo_log(svn_boolean_t f
> if (deleted_rangelist->nelts)
> {
> svn_rangelist__set_inheritance(deleted_rangelist, FALSE);
> - SVN_ERR(svn_rangelist_merge(&master_noninheritable_rangelist,
> - deleted_rangelist,
> - scratch_pool));
> + SVN_ERR(svn_rangelist_merge2(master_noninheritable_rangelist,
> + deleted_rangelist,
> + scratch_pool, iterpool));
> SVN_ERR(svn_rangelist_remove(&master_inheritable_rangelist,
> deleted_rangelist,
> master_inheritable_rangelist,
> @@ -1952,9 +1959,9 @@ svn_client_mergeinfo_log(svn_boolean_t f
> if (finding_merged)
> {
> /* Roll all the merged revisions into one rangelist. */
> - SVN_ERR(svn_rangelist_merge(&master_inheritable_rangelist,
> - master_noninheritable_rangelist,
> - scratch_pool));
> + SVN_ERR(svn_rangelist_merge2(master_inheritable_rangelist,
> + master_noninheritable_rangelist,
> + scratch_pool, scratch_pool));

You can use iterpool as scratch_pool here.
>
> }
> else
> @@ -1970,9 +1977,10 @@ svn_client_mergeinfo_log(svn_boolean_t f
> apr_array_header_t *subtree_merged_rangelist =
> svn__apr_hash_index_val(hi);
>
> - SVN_ERR(svn_rangelist_merge(&source_master_rangelist,
> - subtree_merged_rangelist,
> - iterpool));
> + svn_pool_clear(iterpool);
> + SVN_ERR(svn_rangelist_merge2(source_master_rangelist,
> + subtree_merged_rangelist,
> + scratch_pool, iterpool));
> }
>
> /* From what might be eligible subtract what we know is partially merged
> @@ -1981,9 +1989,9 @@ svn_client_mergeinfo_log(svn_boolean_t f
> master_noninheritable_rangelist,
> source_master_rangelist,
> FALSE, scratch_pool));
> - SVN_ERR(svn_rangelist_merge(&source_master_rangelist,
> - master_noninheritable_rangelist,
> - scratch_pool));
> + SVN_ERR(svn_rangelist_merge2(source_master_rangelist,
> + master_noninheritable_rangelist,
> + scratch_pool, scratch_pool));
> SVN_ERR(svn_rangelist_remove(&master_inheritable_rangelist,
> master_inheritable_rangelist,
> source_master_rangelist,
>

(same) You can use iterpool for the scratch pool here, as that pool hasn't been destroyed at this scope.

And at the very bottom of this function we run a log operation where we could also pass iterpool as a scratch pool if we extend the scope a bit.

        Bert
Received on 2011-07-22 12:30:29 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.