[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Neels J Hofmeyr <neels_at_elego.de>
Date: Sun, 21 Aug 2011 03:17:56 +0200


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?


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.

Received on 2011-08-21 03:18:31 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.