> -----Original Message-----
> From: i...@apache.org [mailto:i...@apache.org]
> Sent: dinsdag 22 april 2014 14:04
> To: comm...@subversion.apache.org
> Subject: svn commit: r1589099 - /subversion/trunk/subversion/libsvn_fs/fs-
> loader.c
> 
> Author: ivan
> Date: Tue Apr 22 12:04:02 2014
> New Revision: 1589099
> 
> URL: http://svn.apache.org/r1589099
> Log:
> * subversion/libsvn_fs/fs-loader.c
>   (svn_fs_open2): Use SCRATCH_POOL for open_fs vtable call.
> 
> Modified:
>     subversion/trunk/subversion/libsvn_fs/fs-loader.c
> 
> Modified: subversion/trunk/subversion/libsvn_fs/fs-loader.c
> URL:
> http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs/fs-
> loader.c?rev=1589099&r1=1589098&r2=1589099&view=diff
> ==========================================================
> ====================
> --- subversion/trunk/subversion/libsvn_fs/fs-loader.c (original)
> +++ subversion/trunk/subversion/libsvn_fs/fs-loader.c Tue Apr 22 12:04:02
> 2014
> @@ -513,7 +513,7 @@ svn_fs_open2(svn_fs_t **fs_p, const char
> 
>    SVN_ERR(fs_library_vtable(&vtable, path, scratch_pool));
>    *fs_p = fs_new(fs_config, result_pool);
> -  SVN_ERR(vtable->open_fs(*fs_p, path, common_pool_lock, result_pool,
> +  SVN_ERR(vtable->open_fs(*fs_p, path, common_pool_lock, scratch_pool,
>                            common_pool));

This either needs a fix somewhere or a comment describing the nonstandard 
passing of pool arguments, as usually a scratch_pool lives shorter than a 
'common_pool', while the usual argument order is 'result_pool, scratch_pool'.

        Bert 


Reply via email to