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 */