Re: [PATCH] why not loop through apr_hash_t with pool as NULL.

Daniel Rall <dlr@collab.net> writes:

>
> + svn_pool_clear(subpool);
> +

The iterator is allocated from subpool, it would need to use pool if
subpool is cleared.

> /* KEY will be the entry name in ancestor (about which we
> simple don't care), VAL the dirent. */
> apr_hash_this(hi, NULL, NULL, &val);
> entry = val;
> nk_args.id = entry->id;
> SVN_ERR(svn_fs_base__retry_txn(fs, txn_body_node_kind, &nk_args,
> - pool));
> + subpool));
> entry->kind = nk_args.kind;
> }
> + svn_pool_destroy(subpool);
> }
> else
> {
>
> This may not provide meaningful memory savings -- how expensive is
> svn_fs_base__retry_txn()?

Not expensive enough for anyone to notice so far :)

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org