[prev in list] [next in list] [prev in thread] [next in thread] 

List:       subversion-issues
Subject:    =?UTF-8?B?W0lzc3VlIDEwOTNdICAnc3ZuIGRpZmYnIGRvZXMgbm90IGhhbmRsZQ==?=
From:       sussman () tigris ! org
Date:       2004-04-30 16:29:16
Message-ID: 20040430162916.5216.qmail () tigris ! org
[Download RAW message or body]

http://subversion.tigris.org/issues/show_bug.cgi?id=1093






------- Additional comments from sussman@tigris.org Fri Apr 30 09:29:15 -0700 2004 -------
I need to put this issue away for a week or so, but here's the next step in this
process, in case jpieper or someone else has time to work on it.

The new history tracing stuff is all in libsvn_client/ra.c.  The new client API
is svn_client__repos_locations(), which has a custom log-receiver func/baton and
uses a helper function svn_client__prev_log_path().

svn_client__prev_log_path() is the current point of interest.  It began life as
a duplicate of code already in 'svn blame'.  The original goal was to gather
this logic into a separate function, so that svn_client__repos_locations() and
svn_client_blame() could *both* make use of it.  Unfortunately, this new
factorized function sat in a patch for a few months, and svn_client_blame's code
evolved separately.

So the task now is to see *how* blame's code changed, and apply those same fixes
to svn_client__prev_log_path().  Then make svn_client_blame use the factorized func.


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

[prev in list] [next in list] [prev in thread] [next in thread]