Yeah, it does seem like a good idea. Committed to trunk in r22909.
On Tue, 02 Jan 2007, Joe Swatosh wrote:
> Hi All,
>
> I found this when I tried to run a debug build. Not sure how
> important it is, but it seems like a good idea.
>
> [[[
> * subversion/bindings/swig/core.i
> Release memory aquired with ruby_xmalloc with ruby_xfree.
> ]]]
>
> Index: subversion/bindings/swig/core.i
> ===================================================================
> --- subversion/bindings/swig/core.i (revision 22877)
> +++ subversion/bindings/swig/core.i (working copy)
> @@ -691,7 +691,7 @@
>
> ~apr_pool_wrapper_t() {
> apr_pool_wrapper_destroy(self);
> - free(self);
> + xfree(self);
> }
> };
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org
- application/pgp-signature attachment: stored
Received on Thu Jan 4 22:20:38 2007