You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Greg Stein <gs...@gmail.com> on 2008/10/16 10:31:47 UTC

Re: svn commit: r33688 - branches/fsfs-pack/subversion/libsvn_fs_fs

On Thu, Oct 16, 2008 at 2:41 AM,  <hw...@tigris.org> wrote:
>...
> +++ branches/fsfs-pack/subversion/libsvn_fs_fs/fs_fs.c  Thu Oct 16 02:41:58 2008        (r33688)
>...
> +packer_func(void *baton,
> +            const char *path,
> +            const apr_finfo_t *finfo,
> +            apr_pool_t *pool)
> +{
>...
> +  /* Copy all the bits from the rev file to the end of the pack file. */
> +  SVN_ERR(svn_stream_open_readonly(&rev_stream, path, pool, pool));
> +  SVN_ERR(svn_stream_copy2(rev_stream, pb->pack_stream, NULL, NULL, pool));
> +  svn_stream_close(rev_stream);
> +
> +  return SVN_NO_ERROR;

svn_stream_close() can return an error. So either toss it with clear()
or return it as the result.

>...

Cheers,
-g

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org