Author: brane
Date: Wed Feb 4 10:38:19 2015
New Revision: 1657111
URL: http://svn.apache.org/r1657111
Log:
On the reuse-ra-session branch: Fix session closing.
* subversion/libsvn_ra/ra_loader.c
(svn_ra__close): Do indeed destroy the session pool.
The assumption that it couldn't be safely destroyed was caused by
a misunderstanding of the side effects of a different bug that
is now fixed.
Modified:
subversion/branches/reuse-ra-session/subversion/libsvn_ra/ra_loader.c
Modified: subversion/branches/reuse-ra-session/subversion/libsvn_ra/ra_loader.c
URL:
http://svn.apache.org/viewvc/subversion/branches/reuse-ra-session/subversion/libsvn_ra/ra_loader.c?rev=1657111&r1=1657110&r2=1657111&view=diff
==============================================================================
--- subversion/branches/reuse-ra-session/subversion/libsvn_ra/ra_loader.c
(original)
+++ subversion/branches/reuse-ra-session/subversion/libsvn_ra/ra_loader.c Wed
Feb 4 10:38:19 2015
@@ -225,14 +225,14 @@ check_ra_version(const svn_version_t *ra
return SVN_NO_ERROR;
}
+/* -------------------------------------------------------------- */
+
+/*** Private Inter-Library Interfaces ***/
void
svn_ra__close(svn_ra_session_t *session)
{
- /* Just clear the pool here. Do not destroy it, because we don't own
- it and we don't know how many references there may be to this
- pool elsewhere. */
if (session && session->pool)
- apr_pool_clear(session->pool);
+ apr_pool_destroy(session->pool);
}
/* -------------------------------------------------------------- */
