You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by iv...@apache.org on 2011/06/28 21:29:34 UTC
svn commit: r1140798 - in /subversion/trunk/subversion/mod_dav_svn:
dav_svn.h repos.c
Author: ivan
Date: Tue Jun 28 19:29:33 2011
New Revision: 1140798
URL: http://svn.apache.org/viewvc?rev=1140798&view=rev
Log:
Remove unused struct member.
* subversion/mod_dav_svn/dav_svn.h
(dav_resource_private): Remove POOL.
* subversion/mod_dav_svn/repos.c
(get_parent_resource, walk): Do not initialize info->pool.
(do_walk): Do not clear info->pool.
Modified:
subversion/trunk/subversion/mod_dav_svn/dav_svn.h
subversion/trunk/subversion/mod_dav_svn/repos.c
Modified: subversion/trunk/subversion/mod_dav_svn/dav_svn.h
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/mod_dav_svn/dav_svn.h?rev=1140798&r1=1140797&r2=1140798&view=diff
==============================================================================
--- subversion/trunk/subversion/mod_dav_svn/dav_svn.h (original)
+++ subversion/trunk/subversion/mod_dav_svn/dav_svn.h Tue Jun 28 19:29:33 2011
@@ -284,9 +284,6 @@ struct dav_resource_private {
/* Cache any revprop change error */
svn_error_t *revprop_error;
- /* Pool to allocate temporary data from */
- apr_pool_t *pool;
-
/* Scratch pool for temporary allocations. This pool created on demand or
cleared if already created in mod_dav callbacks. Currently used only by
insert_liveprop() callback. */
Modified: subversion/trunk/subversion/mod_dav_svn/repos.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/mod_dav_svn/repos.c?rev=1140798&r1=1140797&r2=1140798&view=diff
==============================================================================
--- subversion/trunk/subversion/mod_dav_svn/repos.c (original)
+++ subversion/trunk/subversion/mod_dav_svn/repos.c Tue Jun 28 19:29:33 2011
@@ -2403,7 +2403,6 @@ get_parent_resource(const dav_resource *
parent->uri = get_parent_path(resource->uri, TRUE, resource->pool);
parent->info = parentinfo;
- parentinfo->pool = resource->info->pool;
parentinfo->uri_path =
svn_stringbuf_create(get_parent_path(resource->info->uri_path->data,
TRUE, resource->pool),
@@ -4047,9 +4046,6 @@ do_walk(walker_ctx_t *ctx, int depth)
apr_hash_t *children;
apr_pool_t *iterpool;
- /* Clear the temporary pool. */
- svn_pool_clear(ctx->info.pool);
-
/* The current resource is a collection (possibly here thru recursion)
and this is the invocation for the collection. Alternatively, this is
the first [and only] entry to do_walk() for a member resource, so
@@ -4254,9 +4250,6 @@ walk(const dav_walk_params *params, int
if (ctx.repos_path != NULL)
ctx.info.repos_path = ctx.repos_path->data;
- /* Create a pool usable by the response. */
- ctx.info.pool = svn_pool_create(params->pool);
-
/* ### is the root already/always open? need to verify */
/* always return the error, and any/all multistatus responses */