Re: [PATCH] Optionally use deltas in dumpfiles

On Mar 28, 2004, at 12:58 AM, Greg Hudson wrote:

> The problem: the output of "svnadmin dump" is useful as a
> platform-independent backup format, as a way of transporting new
> revisions across the net, and for other purposes. But for some
> workloads it's too big, because it dumps all the properties and all
> the plaintext for each changed node-revision.

I like this idea a lot.

> +svn_error_t *svn_repos_dump_fs2 (svn_repos_t *repos,
> + svn_stream_t *dumpstream,
> + svn_stream_t *feedback_stream,
> + svn_revnum_t start_rev,
> + svn_revnum_t end_rev,
> + svn_boolean_t incremental,
> + svn_boolean_t use_deltas,
> + svn_cancel_func_t cancel_func,
> + void *cancel_baton,
> + apr_pool_t *pool);

Since this adds a second boolean flag to this function, would it make
sense to switch it to an integer with several bit flags so we can add
further such options in the future without changing the prototype?

-garrett

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Mar 28 20:31:23 2004