On Thu, Jul 30, 2009 at 05:59:23PM +0100, Julian Foad wrote:
> Hi Daniel.
>
> Thanks for your interest.
>
> Issue #3342 <http://subversion.tigris.org/issues/show_bug.cgi?id=3342>
> is not talking about the "G foo1" notifications. It is about the message
> that says
>
> [[[
> Summary of conflicts:
> Text conflicts: 2
> Property conflicts: 1
> Skipped paths: 10
> ]]]
>
> Sorry - that's my fault for not being clear when I wrote it. I've just
> added a note to the issue to clarify this.
>
> To fix this, I would change the notifier function in
> subversion/svn/notify.c so that it still collects the statistics in
> 'nb->text_conflicts' etc., but does NOT call print_conflict_stats() when
> it gets an 'update_completed' or 'merge_completed' notification.
> Instead, make print_conflict_stats() a public function (named
> svn_cl__print_conflict_stats()) so that the top-level update function
> (svn_cl__update()) can call it to print the stats when the whole update
> is finished.
> Then adjust the callers (update, switch and merge) to call
> svn_cl__print_conflict_stats() after finishing the update/switch/merge
> operation.
Why not gather stats entirely inside of libsvn_client so that
other clients can also benefit?
Stefan
Received on 2009-07-30 19:05:48 CEST