Hi devs,
Recall a discussion on this list about a small bug in command line
"svn blame": if blame output contains a revision number larger than 6
characters (i.e. >= 1000000) the columns would not be correctly
aligned (see [1]). I promised in that thread to fix this, as my first
svn hacking exercise. However, finding some free time to do this took
me much longer than I expected/wanted ("a couple of days" has now
become "a couple of months). Sorry about that.
Anyway, moving on, I want to pick up the ball again. I have some
(newbie) questions about the proposed solution (using "end_revnum" to
determine the maximum column width for the revnum).
1) To do this, the blame_receiver callback needs to know the
end_revnum. So I need to extend svn_client_blame_receiver3_t to
include end_revnum as a parameter, right (so it can be passed to the
receiver inside svn_client_blame5 (blame.c))?
2) Since svn_client_blame_receiver3_t was introduced in 1.7 (according
to the comment), can I just change it, or do I still have to introduce
a svn_client_blame_receiver4_t for backward compat?
3) Do I add the end_revnum parameter at the end of the parameter list,
just before the *pool?
Thanks.
[1] http://svn.haxx.se/dev/archive-2010-04/0463.shtml
Cheers,
--
Johan
