Re: svn commit: r1159880 - in /subversion/trunk/subversion/libsvn_client: client.h diff.c merge.c repos_diff.c repos_diff_summarize.c

Julian!

Great stuff! :D

Well, I haven't looked at the patch in great detail, but the summary reads
like honey and cream. Ever since I waded through the diff code years ago
this task has been on my, er, long-term to-do list... Thanks! :)

Oh but it's not done yet, of course. The "Later we can drive...", does that
mean you'll continue to work on it in the coming weeks? Or are you off to
other tasks and it's going to be *a lot* later if it depended on you?

~Neels

On 08/20/2011 03:56 PM, julianfoad_at_apache.org wrote:
> Author: julianfoad
> Date: Sat Aug 20 13:56:52 2011
> New Revision: 1159880
>
> URL: http://svn.apache.org/viewvc?rev=1159880&view=rev
> Log:
> Change the implementation of 'svn diff --summarize' to use the standard
> 'diff callbacks' mechanism, so that it may be more easily extended. The
> summary mode currently supports only repos-repos diffs, and it was reporting
> the summary results directly from a delta editor drive. Instead, write a
> set of diff callbacks that produce summary information, and drive them via
> the existing svn_client__get_diff_editor() for a repos-repos diff. Later we
> can drive the same diff callbacks from WC-WC and WC-repos diff sources.
...